jacek-lewandowski commented on code in PR #2807:
URL: https://github.com/apache/cassandra/pull/2807#discussion_r1362487046
##########
src/java/org/apache/cassandra/cql3/QueryProcessor.java:
##########
@@ -428,11 +455,16 @@ public static Prepared parseAndPrepare(String query,
ClientState clientState, bo
qualifiedStatement.setKeyspace(clientState);
keyspace = qualifiedStatement.keyspace();
}
-
// Note: if 2 threads prepare the same query, we'll live so don't
bother synchronizing
CQLStatement statement = raw.prepare(clientState);
statement.validate(clientState);
+ // Set CQL string for AlterSchemaStatement as this is used to
serialize the transformation
+ // in the cluster metadata log
+ if (statement instanceof AlterSchemaStatement)
+ ((AlterSchemaStatement)statement).setCql(query);
Review Comment:
would it be possible to serialize schema transformation in more formalized
way? I don't know if it is possible to have bound variables in alter schema
statement, but if so, or it will be possible in the future, what would happen
then?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
