k-rus commented on PR #4038: URL: https://github.com/apache/cassandra/pull/4038#issuecomment-3174103050
@netudima thank you for your comments. I addressed them except the empty name is invalid char. This PR focuses on table and keyspace names, thus I don't see a need of adding index name tests. There is a separate issue for dealing with index names, [CASSANDRA-20445](https://issues.apache.org/jira/browse/CASSANDRA-20445). I suggest to address more tests there as it will be more natural. -- 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-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org
