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

Remove duplicated code for starting remote processes in replication tests

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: 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
          knutanders Knut Anders Hatlen added a comment -

          Thanks, Dag. Committed r1344190.

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

          Looks like a safe and good cleanup. +1

          Show
          dagw Dag H. Wanvik added a comment - Looks like a safe and good cleanup. +1
          Hide
          knutanders 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
          knutanders 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:
              knutanders Knut Anders Hatlen
              Reporter:
              knutanders Knut Anders Hatlen
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development