maedhroz commented on code in PR #2433: URL: https://github.com/apache/cassandra/pull/2433#discussion_r1243107657
########## conf/cassandra.yaml: ########## @@ -1956,6 +1956,15 @@ drop_compact_storage_enabled: false # if zero_ttl_on_twcs_enabled is set to false, this property is irrelevant as such statements will fail. #zero_ttl_on_twcs_warned: true +# The default secondary index implementation when CREATE INDEX does not specify one via USING. +# ex. "cassandra" - (default) traditional secondary index, implemented as a hidden table attached to the primary table +# ex. "sai" - storage-attched index, implemented via optimized SSTable/Memtable-attached indexes Review Comment: My only concern w/ this patch was not to break the `CREATE CUSTOM INDEX` DDL that *happens* to still work as a result of how support for custom indexes was implemented. Even if it's possible, we don't want to document it or encourage people to use it. To be honest, I think the `CREATE CUSTOM INDEX` syntax was probably a mistake. -- 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]
