Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-6978

Update qserv for changes in Log interface

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Qserv
    • Labels:

      Description

      DM-6521 improved Log class interface by replacing some static methods with non-static. Qserv is currently using couple of static methods which were retained in Log class for the duration of this migration. Once updated log package is released update qserv code to use new non-static methods and remove static methods from Log class after that.

        Attachments

          Issue Links

            Activity

            Hide
            salnikov Andy Salnikov added a comment -

            Hsin-Fang Chiang, could you do a quick review of my changes to both log and qserv packages, the changes are trivial, should not take long.

            Thanks,
            Andy

            Show
            salnikov Andy Salnikov added a comment - Hsin-Fang Chiang , could you do a quick review of my changes to both log and qserv packages, the changes are trivial, should not take long. Thanks, Andy
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Looks good to me.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Looks good to me.
            Hide
            salnikov Andy Salnikov added a comment -

            Thanks!. Merged both packages.

            Show
            salnikov Andy Salnikov added a comment - Thanks!. Merged both packages.

              People

              • Assignee:
                salnikov Andy Salnikov
                Reporter:
                salnikov Andy Salnikov
                Reviewers:
                Hsin-Fang Chiang
                Watchers:
                Andy Salnikov, Hsin-Fang Chiang
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel