[ 
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

Reply via email to