AMashenkov commented on code in PR #2443:
URL: https://github.com/apache/ignite-3/pull/2443#discussion_r1298620388
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rel/set/IgniteMapSetOp.java:
##########
@@ -66,6 +68,32 @@ default RelDataType buildRowType() {
return builder.build();
}
+ /**
+ * Creates a row type produced by MAP phase of INTERSECT/EXCEPT operator.
+ * For input row (a:type1, b:type2) and {@code inputsNum} = {@code 3} it
produces the following row type:
+ * <pre>
+ * f0: type1
+ * f1: type2
+ * _count_0: int
+ * _count_1: int
+ * _count_2: int
+ * </pre>
+ */
+ public static RelDataType buildRowType(IgniteTypeFactory typeFactory,
RelDataType rowType, int inputsNum) {
Review Comment:
I'd expect to see row layout in javadoc to AbstractSetOpNode.
Here, it just worth to mention that we append few `int` columns for
aggregates needs.
However, you may left it as is.
--
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]
