adelapena commented on code in PR #2673:
URL: https://github.com/apache/cassandra/pull/2673#discussion_r1342862641
##########
src/java/org/apache/cassandra/cql3/statements/SelectStatement.java:
##########
@@ -109,6 +112,9 @@ public class SelectStatement implements
CQLStatement.SingleKeyspaceCqlStatement
private static final NoSpamLogger noSpamLogger =
NoSpamLogger.getLogger(SelectStatement.logger, 1, TimeUnit.MINUTES);
public static final int DEFAULT_PAGE_SIZE = 10000;
+ public static final String TOPK_CONSISTENCY_LEVEL_ERROR = "Top-k queries
can only be run with consistency level one. Consistency level %s was used.";
Review Comment:
I think we are missing the client warning saying that vector indexes are
experimental. That warning message should probably be quite similar
to`SASIIndex.USAGE_WARNING` and `View.USAGE_WARNING`. We could also include
information about limitations. For example:
```
SAI ANN indexes on vector columns are experimental and are not recommended
for production use.
They cannot be queried with a read consistency level higher than
ONE/LOCAL_ONE.
They cannot use paging either, nor be used in aggregation queries.
These limitations are planned to be removed in future versions.
```
--
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]
