Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-7788

Improve PasswordAuthenticator default super user setup

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Low
    • Resolution: Fixed
    • Fix Version/s: 1.2.19, 2.0.10, 2.1.1
    • Component/s: None
    • Labels:
      None

      Description

      Auth#hasExistingUsers() itself can be improved - should do the first SELECT query with CL.ONE first, before trying QUORUM.

      There is no good reason to limit setup to

              if (DatabaseDescriptor.getSeeds().contains(FBUtilities.getBroadcastAddress()) || !DatabaseDescriptor.isAutoBootstrap())
      

      If there was one, it's a leftover from 1.1 private implementation era.

      Finally, the setup write itself should happen at CL.ONE. The logged warning would be a lie in the QUORUM case if at least one node got the write ("skipped default user setup: some nodes were not ready")

        Attachments

        1. 7788.txt
          7 kB
          Aleksey Yeschenko

          Activity

            People

            • Assignee:
              aleksey Aleksey Yeschenko
              Reporter:
              aleksey Aleksey Yeschenko
              Authors:
              Aleksey Yeschenko
              Reviewers:
              Jeremiah Jordan
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: