Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented May 22, 2022

What changes were proposed in this pull request?

Upgrade Parquet version to 1.12.3.

Why are the changes needed?

PARQUET-1968 can simplify current In predicate filter pushdown.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Existing test.

@dongjoon-hyun
Copy link
Member

cc @sunchao

@github-actions github-actions bot added the SQL label May 22, 2022
@wangyum wangyum changed the title [WIP][SQL] Upgrade parquet to 1.12.3 [SPARK-39308][SQL] Upgrade parquet to 1.12.3 May 26, 2022
Copy link
Member

@sunchao sunchao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@huaxingao
Copy link
Contributor

@wangyum I saw 1.12.3 was released 4 hours ago, but somehow I can't find the release notes. Do you have a link to the release notes? Trying to see the list of PRs that are included in 1.12.3.

@wangyum
Copy link
Member Author

wangyum commented May 27, 2022

@huaxingao You can compare two tags: apache/parquet-java@apache-parquet-1.12.2...apache-parquet-1.12.3

@wangyum wangyum changed the title [SPARK-39308][SQL] Upgrade parquet to 1.12.3 [SPARK-39308][BUILD][SQL] Upgrade parquet to 1.12.3 May 27, 2022
@wangyum wangyum closed this in 6d8efb5 May 27, 2022
@wangyum
Copy link
Member Author

wangyum commented May 27, 2022

Thank you all.

@wangyum
Copy link
Member Author

wangyum commented May 27, 2022

Merged to master.

@wangyum wangyum deleted the parquet-1.12.3 branch May 27, 2022 02:26
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you all.

wangyum added a commit that referenced this pull request Jul 4, 2022
### What changes were proposed in this pull request?

This PR updates `FilterPushdownBenchmark` results.

### Why are the changes needed?

These PRs do not or do not fully update the `FilterPushdownBenchmark` results:
#36328
#36629
#36892

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

N/A.

Closes #37022 from wangyum/SPARK-39631.

Authored-by: Yuming Wang <[email protected]>
Signed-off-by: Yuming Wang <[email protected]>
Kimahriman pushed a commit to Kimahriman/spark that referenced this pull request Feb 12, 2023
### What changes were proposed in this pull request?

Upgrade Parquet version to 1.12.3.

### Why are the changes needed?

[PARQUET-1968](https://issues.apache.org/jira/browse/PARQUET-1968) can simplify current In predicate filter pushdown.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Existing test.

Closes apache#36629 from wangyum/parquet-1.12.3.

Authored-by: Yuming Wang <[email protected]>
Signed-off-by: Yuming Wang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants