korlov42 commented on a change in pull request #9095:
URL: https://github.com/apache/ignite/pull/9095#discussion_r640670530
##########
File path:
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/exec/rel/MinusNode.java
##########
@@ -274,94 +66,21 @@ private void addOnSingle(Row row, int setIdx) {
}
Review comment:
I want to put a comment `should we remove the key as far as the first
row comes from setIdx>0 in case all = false?` to the line 271, but GH didn't
allow me to do this since this line is too far from the nearest changed by this
patch. So I put it to the line 274 and left a note describing what line this
intended for (`this comment should be a few rows above the current one (see
line 271).` -- this is the note).
--
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]
