-
Notifications
You must be signed in to change notification settings - Fork 50
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
deps: update dependency com.google.cloud:google-cloud-spanner-bom to v6.27.0 #898
deps: update dependency com.google.cloud:google-cloud-spanner-bom to v6.27.0 #898
Conversation
Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot. |
Hi @olavloite, can you please look into the failing test here? Seems to be related to some recent change in PostgreSQLStatementParser. Thanks!
|
3325661
to
5ffb8d7
Compare
5ffb8d7
to
76016f4
Compare
@olavloite gentle bump - are you able to take a look? Looks like there was a similar issue you handled in the spanner repo: googleapis/java-spanner#1730 |
Some tests used invalid escaping of quotes in PostgreSQL queries. They assumed that a quote in a quoted string could be escaped by prefixing it with a backslash. This is however not allowed in PostgreSQL, and instead the quote must be escaped by including the same quote twice in the string. The Spanner Java client library contained a bug that allowed the backslash-escaped quotes to be accepted. This bug has been fixed in 6.25.7 and these tests therefore need to be modified to be able to update to that version. Fixes the build error in #898
@alicejli @rajatbhatta Sorry, I missed the initial request. I've applied a fix in #909. Once that is merged, a rebase of this PR should make the build failure go away. |
76016f4
to
41b16ca
Compare
41b16ca
to
9bb964b
Compare
Some tests used invalid escaping of quotes in PostgreSQL queries. They assumed that a quote in a quoted string could be escaped by prefixing it with a backslash. This is however not allowed in PostgreSQL, and instead the quote must be escaped by including the same quote twice in the string. The Spanner Java client library contained a bug that allowed the backslash-escaped quotes to be accepted. This bug has been fixed in 6.25.7 and these tests therefore need to be modified to be able to update to that version. Fixes the build error in #898
Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot. |
9bb964b
to
2d016fa
Compare
This PR contains the following updates:
6.25.5
->6.27.0
Release Notes
googleapis/java-spanner
v6.27.0
Compare Source
Features
Dependencies
v6.26.0
Compare Source
Features
Bug Fixes
Dependencies
v6.25.7
Compare Source
Dependencies
v6.25.6
Compare Source
Bug Fixes
Documentation
Dependencies
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.