-
Notifications
You must be signed in to change notification settings - Fork 25.8k
SQL CLI requires strict version matching #36186
Copy link
Copy link
Open
Labels
:Analytics/SQLSQL queryingSQL queryingTeam:AnalyticsMeta label for analytical engine team (ESQL/Aggs/Geo)Meta label for analytical engine team (ESQL/Aggs/Geo)
Description
elasticsearch/x-pack/plugin/sql/sql-cli/src/main/java/org/elasticsearch/xpack/sql/cli/Cli.java
Line 151 in 20e1b5e
| ". This version of CLI only works with Elasticsearch version " + Version.CURRENT.toString()); |
Perhaps this could check and indicate against minimumCompatibilityVersion() instead? It might misindicate if the SQL feature isn't properly represented in the logic, but it would be a step better than checking strict version equality.
cc @kurtado
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
:Analytics/SQLSQL queryingSQL queryingTeam:AnalyticsMeta label for analytical engine team (ESQL/Aggs/Geo)Meta label for analytical engine team (ESQL/Aggs/Geo)
Type
Fields
Give feedbackNo fields configured for issues without a type.