-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
It is not possible to use the spring @Transactional anotation #684
Comments
Is it supposed to work with Spring Boot 2.7 and Spring Data R2DBC 1.5.6.? I don't see it documented as supported. |
I think it should be supported because there is not any documentation saying that it is not supported. How would i know that it does not work before starting using it if it is not documented? If that support is not implemented soon i will have to throw out 4 months of work and rewrite my service with any db that does not comes from google. That is a really bad position that you guys put me. |
It's in the release notes: https://github.com/GoogleCloudPlatform/cloud-spanner-r2dbc/releases |
That information should be on the main project page https://github.com/GoogleCloudPlatform/cloud-spanner-r2dbc and here https://cloud.google.com/spanner/docs/use-oss-r2dbc not "hidden" only on the release notes. Those are what people first look at. But anyway, is there a plan to implement compatibility with spring? And what is the target date for that? |
Looks like a duplicate of #314. |
@burkedavison @meltsufin |
Fixed in #738. |
If i try to use the @transactional annotation from spring i get the error
To reproduce the error you can use the test transactionFailTest that you can find here https://github.com/filipenfst/spanner-context-error/blob/master/src/test/kotlin/com/example/spannercontexterror/R2DBCSpannerErrors.kt
The issue is cause by the begingTransaction method on the SpannerClientLibraryConnection class
The text was updated successfully, but these errors were encountered: