Lucene - Core
  1. Lucene - Core
  2. LUCENE-4603

The test framework should report forked JVM PIDs upon heartbeats

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1, 6.0
    • Component/s: general/test
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      This would help in getting a stack trace of a hung JVM before the timeout and/or in killing the offending JVM.

      RR issue:
      https://github.com/carrotsearch/randomizedtesting/issues/135

        Issue Links

          Activity

          Hide
          Dawid Weiss added a comment -

          I've fixed this in version 2.0.5. It reports forked JVM PID, no stack trace – there's another issue for that but it's more complicated than "just" PIDs. I used the method mentioned by Uwe and dump the entire PID string (VM name, whatever it's going to be). On hotspot this should be PID@host. On other vendor VMs – we'll see.

          I'll update as soon as version 2.0.5 propagates through Maven repositories.

          Show
          Dawid Weiss added a comment - I've fixed this in version 2.0.5. It reports forked JVM PID, no stack trace – there's another issue for that but it's more complicated than "just" PIDs. I used the method mentioned by Uwe and dump the entire PID string (VM name, whatever it's going to be). On hotspot this should be PID@host. On other vendor VMs – we'll see. I'll update as soon as version 2.0.5 propagates through Maven repositories.
          Hide
          Uwe Schindler added a comment -

          I have seen the randomizedtesting/junit4 commit. One addition: Could we also print all pidStrings on startup, too? Sometimes tests don't heartbeat, but I still want to signal them. So along the messages:

          [junit4:junit4] <JUnit4> says aloha! Master seed: 47E240512F1F7A09
          [junit4:junit4] Executing 328 suites with 2 JVMs.
          

          it could print:

          J0: pidString; J1: pidString;...
          
          Show
          Uwe Schindler added a comment - I have seen the randomizedtesting/junit4 commit. One addition: Could we also print all pidStrings on startup, too? Sometimes tests don't heartbeat, but I still want to signal them. So along the messages: [junit4:junit4] <JUnit4> says aloha! Master seed: 47E240512F1F7A09 [junit4:junit4] Executing 328 suites with 2 JVMs. it could print: J0: pidString; J1: pidString;...
          Hide
          Dawid Weiss added a comment -

          Sure, added to the list of issues to fix. I'll respin in a week, perhaps there'll be more fixes by then, ok?

          Show
          Dawid Weiss added a comment - Sure, added to the list of issues to fix. I'll respin in a week, perhaps there'll be more fixes by then, ok?
          Hide
          Commit Tag Bot added a comment -

          [trunk commit] Dawid Weiss
          http://svn.apache.org/viewvc?view=revision&revision=1419261

          LUCENE-4603: Upgrade randomized testing to version 2.0.5: print forked JVM PIDs on heartbeat from hung tests (Dawid Weiss)

          Show
          Commit Tag Bot added a comment - [trunk commit] Dawid Weiss http://svn.apache.org/viewvc?view=revision&revision=1419261 LUCENE-4603 : Upgrade randomized testing to version 2.0.5: print forked JVM PIDs on heartbeat from hung tests (Dawid Weiss)
          Hide
          Commit Tag Bot added a comment -

          [branch_4x commit] Dawid Weiss
          http://svn.apache.org/viewvc?view=revision&revision=1419263

          LUCENE-4603: Upgrade randomized testing to version 2.0.5: print forked JVM PIDs on heartbeat from hung tests (Dawid Weiss)

          Show
          Commit Tag Bot added a comment - [branch_4x commit] Dawid Weiss http://svn.apache.org/viewvc?view=revision&revision=1419263 LUCENE-4603 : Upgrade randomized testing to version 2.0.5: print forked JVM PIDs on heartbeat from hung tests (Dawid Weiss)

            People

            • Assignee:
              Dawid Weiss
              Reporter:
              Dawid Weiss
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development