[email protected] has uploaded this change for review. ( http://gerrit.cloudera.org:8080/24977
Change subject: IMPALA-15447: synthesize Iceberg scan reports on the fast path ...................................................................... IMPALA-15447: synthesize Iceberg scan reports on the fast path When Impala's fast path skips Iceberg planFiles(), no ScanReport is produced, so a deployment forwarding scan reports (previous change) sees nothing for those queries. Synthesize a report from the file descriptors and conjuncts already computed and hand it only to the configured side reporter; the InMemory reporter and query profile are left untouched. The synthesized report is marked impala.synthesized=true so consumers can distinguish it from one Iceberg produced. It carries distinct delete-file counters (positional, equality, DVs) rather than a lumped total, uses the scanned snapshot's schema id (not the table's current schema, which differs for time-travel queries), and runs the filter through ExpressionUtil.sanitize() so literals are redacted, matching what Iceberg does for real reports. Testing: - Added IcebergFastPathReportTest: asserts the fast path emits a report tagged impala.synthesized=true and a non-Iceberg scan emits none (run: mvn test -Dtest=IcebergFastPathReportTest). The test sets metrics-reporter-impl and resets the memoized side reporter, so run it in its own fork. - Ran gerrit-auto-critic (--dryrun): clean. Change-Id: I9c32206da225e127da37506c820cc35ad8a0e1ab Assisted-by: Claude Opus 4.8 (Claude Code) Co-Authored-By: Claude Opus 4.8 <[email protected]> --- M fe/src/main/java/org/apache/impala/planner/IcebergScanPlanner.java A fe/src/test/java/org/apache/impala/planner/IcebergFastPathReportTest.java 2 files changed, 241 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/77/24977/1 -- To view, visit http://gerrit.cloudera.org:8080/24977 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I9c32206da225e127da37506c820cc35ad8a0e1ab Gerrit-Change-Number: 24977 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <[email protected]>
