zstan commented on code in PR #6593:
URL: https://github.com/apache/ignite-3/pull/6593#discussion_r2353067675
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/CacheKey.java:
##########
@@ -41,22 +42,34 @@ public class CacheKey {
private int hashCode = 0;
+ private final ParsedResult parsedResult;
+
/**
* Constructor.
*
* @param catalogVersion Catalog version.
* @param schemaName Schema name.
- * @param query Query string.
+ * @param parsedResult AST with additional info.
* @param contextKey Optional context key to differ queries with and
without/different flags, having an impact on result plan (like
* LOCAL flag)
* @param paramTypes Types of all dynamic parameters, no any type can be
{@code null}.
*/
- public CacheKey(int catalogVersion, String schemaName, String query,
Object contextKey, ColumnType[] paramTypes) {
+ CacheKey(int catalogVersion, String schemaName, ParsedResult parsedResult,
Object contextKey, ColumnType[] paramTypes) {
this.catalogVersion = catalogVersion;
this.schemaName = schemaName;
- this.query = query;
+ this.query = parsedResult.normalizedQuery();
this.contextKey = contextKey;
this.paramTypes = paramTypes;
+
+ this.parsedResult = parsedResult;
+ }
+
+ ParsedResult parsedResult() {
+ return parsedResult;
+ }
Review Comment:
done
--
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]
