Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-2122

Clean up naming of InvalidOpId(), MinimumOpId(), etc

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Trivial
    • Resolution: Unresolved
    • 1.5.0
    • None
    • consensus

    Description

      We have some aliases in opid_util.h/cc called MinimumOpId() and InvalidOpId() that are poorly named. We should consider changing InvalidOpId() to (0,0) and MinimumOpId() to (1,0) or something similar to reflect how we use those values today. Or come up with better naming. The actual, operational minimum OpId that can appear in a WAL is (1,1).

      Attachments

        Activity

          People

            mpercy Mike Percy
            mpercy Mike Percy
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: