JingsongLi commented on PR #886:
URL: https://github.com/apache/paimon-rust/pull/886#issuecomment-5771893638

   Reviewed head f63548c9. Requirement fit: SUPPORTED; implementation: CLEAN. 
The pre-fix PartitionSet could drop a conjunct while DataFusion treated a 
partition-only filter as exact; returning the original predicate for any 
unrepresented node or repeated field restores the entry-level check, and 
manifest pruning remains conservative. I ran `cargo test -p paimon 
table::partition_filter::tests --lib` (11 passed) and `cargo test -p 
paimon-datafusion --test partition_conjunct_pushdown` (3 passed). No actionable 
issue found in this head. FYI #858 also changes `partition_filter.rs`; if it 
lands first, please preserve these contradictory-conjunct SQL regressions when 
reconciling the implementations.


-- 
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]

Reply via email to