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

Improve PasswordAuthenticator default super user setup

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Low
    • Resolution: Fixed
    • 1.2.19, 2.0.10, 2.1.1
    • None
    • 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

            aleksey Aleksey Yeschenko
            aleksey Aleksey Yeschenko
            Aleksey Yeschenko
            Jeremiah Jordan
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: