Derby
  1. Derby
  2. DERBY-5783

Remove duplicated code for starting remote processes in replication tests

    Details

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

      Description

      ReplicationRun has two copies of each of the methods runUserCommand(), runUserCommandRemotely() and processOutput(). One variant takes an id parameter (used only in debug output), and one doesn't. It would be sufficient to have one copy of each of these methods.

      1. d5783-1a.diff
        5 kB
        Knut Anders Hatlen

        Issue Links

          Activity

          Hide
          Knut Anders Hatlen added a comment -

          Thanks, Dag. Committed r1344190.

          Show
          Knut Anders Hatlen added a comment - Thanks, Dag. Committed r1344190.
          Hide
          Dag H. Wanvik added a comment -

          Looks like a safe and good cleanup. +1

          Show
          Dag H. Wanvik added a comment - Looks like a safe and good cleanup. +1
          Hide
          Knut Anders Hatlen added a comment -

          The attached patch replaces two calls to the three-arg runUserCommandRemotely() method with calls to the four-arg variant of the method, and removes three methods that become unused because of the change.

          Replication tests passed with the patch.

          Show
          Knut Anders Hatlen added a comment - The attached patch replaces two calls to the three-arg runUserCommandRemotely() method with calls to the four-arg variant of the method, and removes three methods that become unused because of the change. Replication tests passed with the patch.

            People

            • Assignee:
              Knut Anders Hatlen
              Reporter:
              Knut Anders Hatlen
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development