alex-plekhanov commented on code in PR #12600:
URL: https://github.com/apache/ignite/pull/12600#discussion_r2680945848
##########
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/exec/AbstractCacheColumnsScan.java:
##########
@@ -34,8 +36,8 @@ public abstract class AbstractCacheColumnsScan<Row> extends
AbstractCacheScan<Ro
/** */
protected final RelDataType rowType;
- /** Participating columns. */
- protected final ImmutableBitSet requiredColumns;
+ /** Row field to column mapping. */
+ protected final int[] fieldColMapping;
Review Comment:
Here we have mapping from node row fields to table row columns. We have
identical naming in IndexScan: idxFieldMapping - from index keys to row fields,
fieldIdxMapping - from row fields to index keys.
Maybe just extend the javadoc?
--
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]
