-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
SparkSubmit: Adding propertyfiles option #36164
SparkSubmit: Adding propertyfiles option #36164
Conversation
425c16c
to
07aaf5d
Compare
cc. @ghostp13409 |
Introduce a new parameter 'properties_file' in SparkSubmitHook and SparkSubmitOperator. This allows loading of extra properties from a specified file, defaulting to conf/spark-defaults.conf if nothing is specified. The goal is to provide greater flexibility for Spark configuration. The changes have been tested for the affected files.
Updated the alt text of an image in the example_twitter_README.md file for better accessibility and SEO. The change provides a more detailed description of the screenshot, enhancing understanding for visually impaired users and search engines."
769c92d
to
ca439a6
Compare
@@ -33,7 +33,7 @@ | |||
1. Save Summary to MySQL | |||
|
|||
***Screenshot:*** | |||
<img src="http://i.imgur.com/rRpSO12.png" width="99%"/> | |||
<img src="http://i.imgur.com/rRpSO12.png" width="99%" alt="Dag for Collecting tweets for four users account"/> | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this change related to the main goal of this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this was a pre-commit failure.
as all the images need an alt-text, may be due to addition of this new rule
https://github.com/DavidAnson/markdownlint/blob/main/doc/md045.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in #36169
closes: #34838
Description
There is an option to override the default conf/spark-defaults.conf properties in Spark-submit
We want to have that option in SparkSubmitOperator and Hook as well.