Uploaded image for project: 'Derby'
  1. Derby
  2. DERBY-6400

harness.NetServer method waitFor() is not reflecting timeout values

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 10.10.2.0, 10.11.1.1
    • Component/s: Test
    • Labels:
      None

      Description

      On a slow machine, derbynetclientmats was seen to fail (with no test pass, nor fail) with the following error:

      org.apache.derby.drda.NetworkServerControl start
      Attempt to shutdown framework: DerbyNetClient
      Server Process did not complete in time. Destroying...
      Server Process did not complete in time. Destroying...
      Exception in thread "main" java.io.IOException: Interrupted system call
      at java.io.BufferedInputStream.fill(BufferedInputStream.java:235)
      at java.io.BufferedInputStream.read1(BufferedInputStream.java:275)
      at java.io.BufferedInputStream.read(BufferedInputStream.java:334)
      at org.apache.derbyTesting.functionTests.harness.BackgroundStreamSaver.run(BackgroundStreamSaver.java:51)
      at java.lang.Thread.run(Thread.java:853)

      It seems to me that the problem is here that in functionTests.harness.NetServer the waitFor() method has a hard value of 60 in it. Instead, this value should be configurable.
      In functionTests.harness.RunTest the waitFor() method uses
      TimedProcess tp = new TimedProcess(pr);
      tp.waitFor(timeout*60);
      It seems NetServer should also be able to check on the timeout value.

        Attachments

        1. DERBY-6400.diff
          3 kB
          Myrna van Lunteren

          Activity

            People

            • Assignee:
              myrna Myrna van Lunteren
              Reporter:
              myrna Myrna van Lunteren
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: