[ https://issues.apache.org/jira/browse/CASSANDRA-14866?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16678001#comment-16678001 ]
Andrés de la Peña commented on CASSANDRA-14866: ----------------------------------------------- [~jjirsa] We don't really need the client warning on {{SELECT}}, probably the warning on {{CREATE INDEX}} is enough. I'm not so sure about the usefulness of the {{cassandra.yaml}} flag, but indeed it would be more aligned with the MV behaviour. Here is an alternative version of the patch warning only on {{CREATE INDEX}}, and with a new {{enable_sasi_indexes}} property in {{cassandra.yaml}} (enabled by default): ||[3.11|https://github.com/adelapena/cassandra/tree/CASSANDRA-14866-mv-3.11]||[trunk|https://github.com/adelapena/cassandra/tree/CASSANDRA-14866-mv-trunk]|| [Here|https://github.com/adelapena/cassandra-dtest/commit/f0eeb1140678f3fe48129441b6dde4b36f8901eb] are a couple of dtests for the warnings and the property. These are our very first SASI dtests so I have put them in a separate {{sasi_test.py}} file. [~snazy] WDYT? > Issue a CQL native protocol warning if SASI indexes are enabled on a table > -------------------------------------------------------------------------- > > Key: CASSANDRA-14866 > URL: https://issues.apache.org/jira/browse/CASSANDRA-14866 > Project: Cassandra > Issue Type: Bug > Components: sasi > Reporter: Andrés de la Peña > Assignee: Andrés de la Peña > Priority: Major > Fix For: 4.0, 3.11.x, 4.x > > > If someone enables SASI indexes then we should return a native protocol > warning that will be printed by cqlsh saying that they are beta quality still > and you need to be careful with using them in production. > This is motivated not only by [the existing bugs and > limitations|https://issues.apache.org/jira/browse/CASSANDRA-12674?jql=project%20%3D%20CASSANDRA%20AND%20status%20%3D%20Open%20AND%20component%20%3D%20sasi] > but for the fact that they haven't been extensively tested yet. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org