WegdanGhazi commented on code in PR #3436:
URL: https://github.com/apache/calcite/pull/3436#discussion_r1334331194
##########
core/src/main/java/org/apache/calcite/prepare/Prepare.java:
##########
@@ -293,10 +293,12 @@ public PreparedResult prepareSql(
root = root.withRel(decorrelate(sqlToRelConverter, sqlQuery, root.rel));
}
- // Trim unused fields.
- root = trimUnusedFields(root);
+ if (configHolder.get().isTrimUnusedFields()) {
+ // Trim unused fields.
+ root = trimUnusedFields(root);
- Hook.TRIMMED.run(root.rel);
Review Comment:
There's no change in the code itself, I've only moved this logic inside a
condition based on the config that determined whether or not to trim unused
fields
--
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]
