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:
[email protected]
