Skip to content

NGSOK-1401 Backport SPARK-48392 (--load-spark-defaults) into spark 3.5.4#25

Merged
Asmoday merged 4 commits into
3.5.4-developfrom
feature/NGSOK-1401
Mar 17, 2026
Merged

NGSOK-1401 Backport SPARK-48392 (--load-spark-defaults) into spark 3.5.4#25
Asmoday merged 4 commits into
3.5.4-developfrom
feature/NGSOK-1401

Conversation

@giggsoff

Copy link
Copy Markdown
Collaborator

What changes were proposed in this pull request?

Backport SPARK-48392 (--load-spark-defaults) into spark 3.5.4.

Why are the changes needed?

To allow store user-provided options in file and merge them with defaults.

Does this PR introduce any user-facing change?

Adds --load-spark-defaults option.

In addition, a property file with Spark configurations can be passed to `bin/spark-submit` via
`--properties-file` parameter. When this is set, Spark will no longer load configurations from
`conf/spark-defaults.conf` unless another parameter `--load-spark-defaults` is provided.

How was this patch tested?

Using unit tests.

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

No.

Signed-off-by: Petr Fedchenkov <giggsoff@gmail.com>
Signed-off-by: Petr Fedchenkov <giggsoff@gmail.com>
Signed-off-by: Petr Fedchenkov <giggsoff@gmail.com>
@giggsoff giggsoff self-assigned this Mar 16, 2026
@giggsoff

Copy link
Copy Markdown
Collaborator Author

We also need to verify that #2 works as expected.

@giggsoff giggsoff requested a review from Asmoday March 16, 2026 10:10
@giggsoff giggsoff marked this pull request as ready for review March 16, 2026 10:11
Signed-off-by: Petr Fedchenkov <giggsoff@gmail.com>
@Asmoday Asmoday merged commit ff4f3b6 into 3.5.4-develop Mar 17, 2026
12 of 15 checks passed
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