Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Log4j2 to Spark #155

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mohamed-ali
Copy link

This changes allows users to introduce log configuration using spark log4j2. This is important because Spark 3.3 onwards migrate from Log4j 1.x, and that Log4j 1.x has reached end of life.

Below a snippet from Spark core migration guide:

Since Spark 3.3, Spark migrates its log4j dependency from 1.x to 2.x because log4j 1.x has reached end of life and is no longer supported by the community. Vulnerabilities reported after August 2015 against log4j 1.x were not checked and will not be fixed. Users should rewrite original log4j properties files using log4j2 syntax (XML, JSON, YAML, or properties format). Spark rewrites the conf/log4j.properties.template which is included in Spark distribution, to conf/log4j2.properties.template with log4j2 properties format.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This changes allows users to introduce log configuration using spark log4j2. This is important because Spark 3.3 onwards migrate from Log4j 1.x, and that Log4j 1.x has reached end of life.

Below a snippet from [Spark core migration guide](https://spark.apache.org/docs/latest/core-migration-guide.html#upgrading-from-core-32-to-33):

> Since Spark 3.3, Spark migrates its log4j dependency from 1.x to 2.x because log4j 1.x has reached end of life and is no longer supported by the community. Vulnerabilities reported after August 2015 against log4j 1.x were not checked and will not be fixed. Users should rewrite original log4j properties files using log4j2 syntax (XML, JSON, YAML, or properties format). Spark rewrites the conf/log4j.properties.template which is included in Spark distribution, to conf/log4j2.properties.template with log4j2 properties format.
@mohamed-ali mohamed-ali changed the title Add support for Log4j2 for Spark Add support for Log4j2 to Spark Jul 23, 2024
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.

1 participant