Details

    • Type: Technical task
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: log
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      DB_S15_05, DB_S15_06
    • Team:
      Data Access and Database

      Attachments

        Issue Links

          Activity

          Hide
          jammes Fabrice Jammes added a comment -

          Code is in Qserv and log repos.

          Show
          jammes Fabrice Jammes added a comment - Code is in Qserv and log repos.
          Hide
          salnikov Andy Salnikov added a comment -

          I don't think any of the changes on this ticket are needed. We should discuss this at the meeting tomorrow.

          Show
          salnikov Andy Salnikov added a comment - I don't think any of the changes on this ticket are needed. We should discuss this at the meeting tomorrow.
          Hide
          jammes Fabrice Jammes added a comment - - edited

          Hi @Andy,

          Ok, I've removed the API code, but keep:

          • unit tests related to log PID (also for child process),
          • other minor improvment to manage lines number in expected log messages.
          • documentation

          Would you agree to review this one ticket please?

          Show
          jammes Fabrice Jammes added a comment - - edited Hi @Andy, Ok, I've removed the API code, but keep: unit tests related to log PID (also for child process), other minor improvment to manage lines number in expected log messages. documentation Would you agree to review this one ticket please?
          Hide
          salnikov Andy Salnikov added a comment -

          Fabrice, it looks mostly OK. I do not think we need any changes in qserv but log looks fine, with one minor comment.

          Show
          salnikov Andy Salnikov added a comment - Fabrice, it looks mostly OK. I do not think we need any changes in qserv but log looks fine, with one minor comment.

            People

            Assignee:
            jammes Fabrice Jammes
            Reporter:
            jammes Fabrice Jammes
            Reviewers:
            Andy Salnikov
            Watchers:
            Andy Salnikov, Fabrice Jammes, Jacek Becla
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: