[GitHub] [cassandra] bereng commented on a change in pull request #1422: modify alter statements with IF EXISTS and IF NOT EXISTS

[GitBox <gi...@apache.org>]

bereng commented on a change in pull request #1422:
URL: https://github.com/apache/cassandra/pull/1422#discussion_r826757543



##########
File path: pylib/cqlshlib/cql3handling.py
##########
@@ -1443,7 +1443,7 @@ def alter_type_field_completer(ctxt, cass):
                               ( "SUPERUSER" | "NOSUPERUSER" )?
                         ;
 
-<alterUserStatement> ::= "ALTER" "USER" <username>
+<alterUserStatement> ::= "ALTER" "USER" ("IF" "EXISTS")? <username>

Review comment:
       Despite the original intent of the ticket mentioning only `ALTER` if we're going to touch `ALTER USER` maybe we should also do `DROP USER` and friends, wdyt @ekaterinadimitrova2 @blerer ? Otherwise it feels like this is only half way through to me.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org