Skip to content

Conversation

@owen6314
Copy link

@owen6314 owen6314 commented Nov 9, 2025

Following up on this PR and move it to Spark 4.0.

Adds support to allow custom snapshot properties to be specified in the Spark session configuration. This allows users to add custom snapshot properties even when running spark SQL DML such as DELETE or MERGE INTO, which was not previously supported.

@github-actions github-actions bot added the spark label Nov 9, 2025
@@ -396,6 +396,37 @@ public void testSparkConfOverride() {
}
}

@TestTemplate
public void testExtraSnapshotMetadataReflectsSessionConfig() {
Copy link
Contributor

Choose a reason for hiding this comment

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

can we add a E2E test which verifies that when this is set the snapshot created actually persists the given property as snapshot property ?

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for the comment (and sorry for the late reply). Added a E2E test testExtraSnapshotMetadataPersistedOnWrite for this purpose.

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.

3 participants