Skip to content

Conversation

@LuciferYang
Copy link
Contributor

@LuciferYang LuciferYang commented Sep 19, 2022

What changes were proposed in this pull request?

This pr aims to upgrade log4j2 from 2.18.0 to 2.19.0.

Why are the changes needed?

Log4j 2.19.0 contains new features and fixes, and this version begin to support for slf4j2:

all changes can be found in the latest changes report.

Does this PR introduce any user-facing change?

No, log4j 2.19.0 maintains binary compatibility with previous releases.

How was this patch tested?

Pass GitHub Actions

@LuciferYang
Copy link
Contributor Author

LuciferYang commented Sep 19, 2022

Test first, will update pr description later

@github-actions github-actions bot added the BUILD label Sep 19, 2022
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.

cc @viirya

@viirya
Copy link
Member

viirya commented Sep 19, 2022

Thanks. Merging to master.

@viirya viirya closed this in 2d6d5e2 Sep 19, 2022
@LuciferYang
Copy link
Contributor Author

thanks @viirya @srowen @dongjoon-hyun

LuciferYang added a commit to LuciferYang/spark that referenced this pull request Sep 20, 2022
### What changes were proposed in this pull request?
This pr aims to upgrade log4j2 from 2.18.0 to 2.19.0.

### Why are the changes needed?
Log4j 2.19.0 contains new features and fixes, and this version begin to support for slf4j2:

- https://issues.apache.org/jira/browse/LOG4J2-3583
- https://issues.apache.org/jira/browse/LOG4J2-2975

all changes can be found in the latest [changes report](https://logging.apache.org/log4j/2.x/changes-report.html#a2.19.0).

### Does this PR introduce _any_ user-facing change?
No, log4j 2.19.0 maintains binary compatibility with previous releases.

### How was this patch tested?
Pass GitHub Actions

Closes apache#37926 from LuciferYang/SPARK-40484.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants