Skip to content

[SPARK-56383][SQL][TEST] Add tests for partition filter extraction from mixed predicates - #55248

Closed
szehon-ho wants to merge 2 commits into
apache:masterfrom
szehon-ho:partition_predicate_from_mixed
Closed

szehon-ho wants to merge 2 commits into
apache:masterfrom
szehon-ho:partition_predicate_from_mixed

Conversation

@szehon-ho

@szehon-ho szehon-ho commented Apr 7, 2026 •

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Add tests to DataSourceV2EnhancedPartitionFilterSuite covering the case where getPartitionFiltersAndDataFilters (called from PushDownUtils.pushPartitionPredicates) extracts additional partition filters from predicates that reference both partition and data columns via extractPredicatesWithinOutputSet.

This was one of the goals of SPARK-55596

Why are the changes needed?

Test coverage

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Ran tests

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude (Cursor)

@szehon-ho
szehon-ho force-pushed the partition_predicate_from_mixed branch 3 times, most recently from 1c6d80d to 3286bf7 Compare April 8, 2026 05:31
@cloud-fan cloud-fan changed the title [SPARK-56383][SQL] Add tests for partition filter extraction from mixed predicates [SPARK-56383][SQL][TEST] Add tests for partition filter extraction from mixed predicates Apr 8, 2026
…ed predicates

Add tests to DataSourceV2EnhancedPartitionFilterSuite covering the case
where getPartitionFiltersAndDataFilters extracts additional partition
filters from predicates that reference both partition and data columns
(via extractPredicatesWithinOutputSet).
@szehon-ho
szehon-ho force-pushed the partition_predicate_from_mixed branch from 3286bf7 to d56c2d3 Compare April 11, 2026 23:46
@cloud-fan

Copy link
Copy Markdown
Contributor

thanks, merging to master!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants