Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-14476

JSTests::invokeLogError log check is still flawed

    • Type: Icon: Bug Bug
    • Resolution: Done
    • Priority: Icon: Major - P3 Major - P3
    • 2.7.4
    • Affects Version/s: None
    • Component/s: Testing Infrastructure
    • None
    • ALL

      Follow-up to SERVER-13460. I think the solution has a race condition that prevents it from working correctly as the durThread may have already decided to do a DurParanoid run but not yet logged. Also we set DurParanoid to 0 after we attach the new appender.

      Rather than fix the race condition, I'd suggest removing the code to futz with durOptions and instead just make the Tee only set _logged to true if logging from the same thread as that will solve the problem for all asynchronous logging, not just this one special case.

      Associated build failure: https://mci.10gen.com/ui/task/mongodb_mongo_master_windows_64_debug_15b29e47fc909680de31c77513d3ab9a49738be0_14_07_07_08_27_06_dbtest_windows_64_debug

      014-07-07T09:39:50.978+0000 [testsuite] going to run suite: js
      2014-07-07T09:39:50.979+0000 [testsuite] 	 going to run test: class JSTests::BuiltinTests
      2014-07-07T09:39:50.979+0000 [testsuite] 	 going to run test: class JSTests::BasicScope
      2014-07-07T09:39:51.020+0000 [testsuite] 	 going to run test: class JSTests::ResetScope
      2014-07-07T09:39:51.020+0000 [testsuite] 	 going to run test: class JSTests::FalseTests
      2014-07-07T09:39:51.055+0000 [testsuite] 	 going to run test: class JSTests::SimpleFunctions
      2014-07-07T09:39:51.105+0000 [testsuite] 	 going to run test: class JSTests::ExecLogError
      2014-07-07T09:39:51.131+0000 [journal] DurParanoid map check 60ms for 160MB
      2014-07-07T09:39:51.159+0000 [testsuite] FAIL: class JSTests::ExecLogError	Expected: !_logger.logged() @src\mongo\dbtests\jstests.cpp:196
      2014-07-07T09:39:51.159+0000 [testsuite] 	 going to run test: class JSTests::InvokeLogError
      2014-07-07T09:39:51.197+0000 [testsuite] ReferenceError: notAFunction is not defined
      

            Assignee:
            benety.goh@mongodb.com Benety Goh
            Reporter:
            mathias@mongodb.com Mathias Stearn
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: