Derby
  1. Derby
  2. DERBY-5617

Improve process handling in SpawnedProcess

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 10.9.1.0
    • Fix Version/s: 10.8.3.0, 10.9.1.0
    • Component/s: Test
    • Labels:
      None

      Description

      SpawnedProcess wraps a java.lang.Process and performs various tasks related to process handling. Using external processes from Java is somewhat tricky, in part because of sub-optimal behavior in the methods provided by Process.

      I plan to implement the following improvements:
      a) Kill the process after a timeout (the clock starts ticking when you create the SpawnedProcess instance).
      b) Don't let interrupts stop SpawnedProcess from waiting for / terminating the process.
      c) Clean up the Process properly.

      1. derby-5617-3a-timer_as_daemon_thread.diff
        0.8 kB
        Kristian Waagan
      2. derby-5617-2a-join_destroy_ordering.diff
        2 kB
        Kristian Waagan
      3. derby-5617-1a-spawnedprocess_improvements.diff
        21 kB
        Kristian Waagan

        Activity

        Kristian Waagan created issue -
        Kristian Waagan made changes -
        Field Original Value New Value
        Attachment derby-5617-1a-spawnedprocess_improvements.diff [ 12514629 ]
        Kristian Waagan made changes -
        Fix Version/s 10.9.0.0 [ 12316344 ]
        Kristian Waagan made changes -
        Kristian Waagan made changes -
        Kristian Waagan made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 10.8.2.3 [ 12318540 ]
        Resolution Fixed [ 1 ]
        Kristian Waagan made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Kathey Marsden made changes -
        Fix Version/s 10.8.2.3 [ 12318540 ]
        Fix Version/s 10.8.3.0 [ 12323456 ]
        Gavin made changes -
        Workflow jira [ 12653381 ] Default workflow, editable Closed status [ 12796843 ]

          People

          • Assignee:
            Kristian Waagan
            Reporter:
            Kristian Waagan
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development