Skip to content

Conversation

@rayudu3745
Copy link
Contributor

@rayudu3745 rayudu3745 commented Dec 18, 2025

Spanner doesn’t have support for local or global temp tables so we have use PersistentTempTable strategy for bulk operations .
SpannerTemporaryTableExporter syntax provided for create table is compatible for Spanner as spanner requires primary key declaration to be outside the column declarations .
this change provides SpannerTemporaryTableExporter which will be used by spanner for DDL of temp tables


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


https://hibernate.atlassian.net/browse/HHH-20003

Copy link
Member

@beikov beikov left a comment

Choose a reason for hiding this comment

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

LGTM. Any reason why you marked it as draft?
Also, does Spanner support insert/update/delete statements in the with clause like PostgreSQL also known as "modifiable CTEs"? If so, we could configure the Spanner dialect to use that strategy instead for multi-table mutations/inserts.

@rayudu3745
Copy link
Contributor Author

Spanner doesn't not support insert/update/delete statements in the with clause, so can't use CteStrategy
PersistantTable strategy might have performance issue with Spanner so put this in draft to check with our team to whether enable this or just fail

@rayudu3745 rayudu3745 marked this pull request as ready for review December 18, 2025 11:05
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.

3 participants