zhaorongsheng commented on PR #67725: URL: https://github.com/apache/doris/pull/67725#issuecomment-5600739149
Thanks for the detailed review. Addressed in the latest head: - Fixed the lambda capture compile failure in `PruneFileScanPartition`; the FE build now succeeds with UI disabled. - Replaced identity-only deferred handling with an explicit `SelectedPartitions` state and represent unknown total partition count as unknown in EXPLAIN rather than a synthetic count. - Preserved no-filter Hive full-scan batch eligibility by materializing the deferred partition state before the batch-mode gate. - Added regression coverage for deferred materialization, batch eligibility, unknown totals, and zero-prune semantics. The relevant FE tests pass (41 tests), and the Hive metadata pruning test passes (16 tests). The duplicate connector partition-pruning RPC is intentionally left out of this focused fix and is tracked in #67739. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
