Skip to content

Conversation

@fedimser
Copy link
Contributor

@fedimser fedimser commented Oct 31, 2025

What changes were proposed in this pull request?

Change error message when user uses TransformWithState with state store provider other than RocksDB state store provider.

Why are the changes needed?

Improves user experience by making it clear that they need to use RocksDBStateStoreProvider.

Does this PR introduce any user-facing change?

Yes, changes error message when user uses TransformWithState with HDFSBackedStateStoreProvider.

Old error message: [UNSUPPORTED_FEATURE.STATE_STORE_MULTIPLE_COLUMN_FAMILIES] The feature is not supported: Creating multiple column families with HDFSBackedStateStoreProvider is not supported.

New error message: [UNSUPPORTED_FEATURE.STORE_BACKEND_NOT_SUPPORTED_FOR_TWS] The feature is not supported: Store backend HDFSBackedStateStoreProvider is not supported by TransformWithState operator. Please use RocksDBStateStoreProvider.

How was this patch tested?

Unit tests: TransformWithStateSuite, TransformWithStateValidationSuite.

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

No.

Copy link
Contributor

@huanliwang-db huanliwang-db left a comment

Choose a reason for hiding this comment

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

lgtm

@HyukjinKwon HyukjinKwon changed the title [SPARK-54117] Throw better error to indicate that TWS is only supported with RocksDB state store provider [SPARK-54117][SS] Throw better error to indicate that TWS is only supported with RocksDB state store provider Nov 2, 2025
Copy link
Contributor

@HeartSaVioR HeartSaVioR left a comment

Choose a reason for hiding this comment

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

+1

@HeartSaVioR
Copy link
Contributor

@fedimser
Could you please rebase with master again? Let's make sure CI passes. Thanks!

@HeartSaVioR
Copy link
Contributor

Thanks! Merging to master.

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.

3 participants