Skip to content

Spark 4.1, 4.0, 3.5: Rename streaming merge-append write option and backport #17347 to 3.5/4.0 - #17403

Open
amogh-jahagirdar wants to merge 1 commit into
apache:mainfrom
amogh-jahagirdar:spark-streaming-merge-append-rename
Open

Spark 4.1, 4.0, 3.5: Rename streaming merge-append write option and backport #17347 to 3.5/4.0#17403
amogh-jahagirdar wants to merge 1 commit into
apache:mainfrom
amogh-jahagirdar:spark-streaming-merge-append-rename

Conversation

@amogh-jahagirdar

@amogh-jahagirdar amogh-jahagirdar commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

This change includes 2 parts:

1.) Rename streaming-merge-append-enabled to use-merge-append-for-streaming for better readability of the conf. This change was just merged and not released so no compatibility issues in changing this config.

2.) After the rename of the config, clean backport of the change from #17347 to Spark 4.0/3.5

@amogh-jahagirdar amogh-jahagirdar changed the title Spark 4.1, 4.0, 3.5: Rename streaming merge-append write option and port to 3.5/4.0 Spark 4.1, 4.0, 3.5: Rename streaming merge-append write option and backport #17347 to 3.5/4.0 Jul 28, 2026
@amogh-jahagirdar

Copy link
Copy Markdown
Contributor Author

cc @dramaticlly

@amogh-jahagirdar

amogh-jahagirdar commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

actually looks like the existing convention is fooFeatureEnabled , some use "use-". I like the present tense a bit better and there's precedence like use-table-distribution-and-ordering

@amogh-jahagirdar
amogh-jahagirdar force-pushed the spark-streaming-merge-append-rename branch 3 times, most recently from e5d8e95 to 6bf9d65 Compare July 29, 2026 00:09
Renames the streaming merge-append write config to
`use-merge-append-for-streaming` in Spark 4.1, and ports the option to
the Spark 3.5 and 4.0 modules.

When enabled, streaming appends use a merge append instead of a fast
append so manifests are merged on commit rather than accumulating as
small manifests.
@amogh-jahagirdar
amogh-jahagirdar force-pushed the spark-streaming-merge-append-rename branch from 6bf9d65 to 464bff6 Compare July 29, 2026 00:09
@amogh-jahagirdar

Copy link
Copy Markdown
Contributor Author

Since the 4.1 change is just a rename that we don't need to worry about compatibility about and the change is fairly small, I included it with the Spark version backports as well.

@danielcweeks danielcweeks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@kevinjqliu kevinjqliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

5 participants