NSAmelchev commented on code in PR #11521:
URL: https://github.com/apache/ignite/pull/11521#discussion_r2012881889


##########
modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/sql/GridSqlQueryParser.java:
##########
@@ -826,11 +826,6 @@ private GridSqlMerge parseMerge(Merge merge) {
 
         res.columns(cols);
 
-        if (!F.isEmpty(MERGE_KEYS.get(merge))) {
-            log.warning("The search row by explicit KEY isn't supported. The 
primary key is always used to search row " +

Review Comment:
   @liyuj  The user should still be warned if they use explicit keys other than 
the primary key. See the `testCheckKeysWarning` test. If no keys are set, the 
warning is redundant.



-- 
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: notifications-unsubscr...@ignite.apache.org

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

Reply via email to