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

Implement globally unique queryId

    XMLWordPrintable

    Details

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

      Attachments

        Issue Links

          Activity

          Hide
          jbecla Jacek Becla added a comment -

          consider reusing queryId from QMeta.

          Show
          jbecla Jacek Becla added a comment - consider reusing queryId from QMeta.
          Hide
          jbecla Jacek Becla added a comment -

          There is a long discussion in DM-853 related to this story.

          Show
          jbecla Jacek Becla added a comment - There is a long discussion in DM-853 related to this story.
          Hide
          jgates John Gates added a comment -

          At the meeting yesterday, I believe it was agreed to use a uin64 for the unique query id. It has the advantage of not needing much storage and is easy to search for. The downside is that it requires a central source to guarantee unique ids, which is expected to be the same source as CSS information. Please, comment on this if you disagree with using uint64.

          Show
          jgates John Gates added a comment - At the meeting yesterday, I believe it was agreed to use a uin64 for the unique query id. It has the advantage of not needing much storage and is easy to search for. The downside is that it requires a central source to guarantee unique ids, which is expected to be the same source as CSS information. Please, comment on this if you disagree with using uint64.
          Hide
          salnikov Andy Salnikov added a comment -

          PR?

          Show
          salnikov Andy Salnikov added a comment - PR?
          Hide
          salnikov Andy Salnikov added a comment -

          Looks OK, see my comments in PR.

          Show
          salnikov Andy Salnikov added a comment - Looks OK, see my comments in PR.
          Hide
          jbecla Jacek Becla added a comment -

          wow, that was quick, it is implemented and reviewed already?

          Show
          jbecla Jacek Becla added a comment - wow, that was quick, it is implemented and reviewed already?

            People

            Assignee:
            jgates John Gates
            Reporter:
            fritzm Fritz Mueller
            Reviewers:
            Andy Salnikov
            Watchers:
            Andy Salnikov, Jacek Becla, John Gates
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: