[GitHub] [ignite] korlov42 commented on a change in pull request #8951: IGNITE-14451 fix corrupt PK due to PK's fields order

[GitBox <gi...@apache.org>]

korlov42 commented on a change in pull request #8951:
URL: https://github.com/apache/ignite/pull/8951#discussion_r607037368



##########
File path: modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/CommandProcessor.java
##########
@@ -1058,8 +1059,8 @@ private IgniteSQLException convert(SchemaOperationException e) {
      * Convert this statement to query entity and do Ignite specific sanity checks on the way.
      * @return Query entity mimicking this SQL statement.
      */
-    private static QueryEntity toQueryEntity(GridSqlCreateTable createTbl) {
-        QueryEntity res = new QueryEntity();
+    private static QueryEntity toQueryEntity(GridKernalContext ctx, GridSqlCreateTable createTbl) {

Review comment:
       seems `ctx` is not used anywhere, so it's better to not to add this




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org