-
Notifications
You must be signed in to change notification settings - Fork 128
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
feat: add PG OID support #2736
Merged
Merged
feat: add PG OID support #2736
Changes from 51 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
6b3f572
feat: add PG OID support
tlhquynh 929abf4
chore: fix lint errors
tlhquynh 4e7ea9d
Resolved merge conflict from branch 'main' into pg-oid.
tlhquynh 0489d9a
Merge branch 'main' into pg-oid
tlhquynh e5a39c5
Update PG.OID implementation according to recent changes.
tlhquynh 537de96
Update PG.OID implementation according to recent changes.
tlhquynh f9ea807
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] 4cb354a
chore: keep session pool ordering when pinging (#2695)
olavloite e505b9e
deps: update dependency com.google.cloud:google-cloud-monitoring to v…
renovate-bot 0184554
feat: allow attempt direct path xds via env var (#2950)
HailongWen 1d756a3
build(deps): update dependency org.apache.maven.plugins:maven-compile…
renovate-bot f6193f9
build(deps): update dependency org.apache.maven.plugins:maven-assembl…
renovate-bot aed1223
deps: update dependency com.google.cloud:sdk-platform-java-config to …
renovate-bot bf2361b
refactor: move skip methods to abstract parser (#2948)
olavloite 57edce9
fix: return type of max commit delay option. (#2953)
arpan14 d7ba42b
refactor: generalize skip methods (#2949)
olavloite 50e2887
perf: keep comments when searching for params (#2951)
olavloite 6258293
chore: randomize session pool order based on TPS (#2792)
olavloite d94922f
chore(main): release 6.62.0 (#2940)
release-please[bot] bd82e0c
chore(main): release 6.62.1-SNAPSHOT (#2957)
release-please[bot] a0a2e8d
chore(deps): update dependency com.google.cloud:google-cloud-spanner …
renovate-bot eaa05da
chore: add session pool options for multiplexed session. (#2960)
arpan14 8e4ee03
deps: update dependency com.google.cloud:google-cloud-trace to v2.38.…
renovate-bot ebf8723
chore: add new members in SessionImpl for multiplexed session. Add a …
arpan14 358e8d3
Update .gitignore to remove IDE specific files and remove unnecessary…
tlhquynh 1e99310
Remove PG.OID external getters.
tlhquynh 039079e
chore: generalise session pool class for multiplexed session. (#2964)
arpan14 b21e262
Merge branch 'main' into pg-oid
tlhquynh 6f971a7
Merge branch 'main' into pg-oid
tlhquynh 5df12ca
chore: emove unnecessary debug.
tlhquynh 4b12761
chore: add multiplexed session implementations for CachedSession/Sess…
arpan14 60712bd
Remove internal PG.OID getters.
tlhquynh 19fbdfb
deps: update dependency com.google.cloud:google-cloud-monitoring to v…
renovate-bot f3f6113
chore(main): release 6.62.1 (#2968)
release-please[bot] 8ac2a16
chore(main): release 6.62.2-SNAPSHOT (#2983)
release-please[bot] 231273e
feat: add support for transaction-level exclusion from change streams…
dengwe1 80ade2e
deps: update dependency com.google.cloud:google-cloud-trace to v2.39.…
renovate-bot ecb87e7
deps: update dependency commons-io:commons-io to v2.16.0 (#2986)
renovate-bot 7444b29
deps: update dependency com.google.cloud:google-cloud-monitoring to v…
renovate-bot 8a9584b
chore(deps): update dependency com.google.cloud:libraries-bom to v26.…
renovate-bot f103f18
chore(main): release 6.63.0 (#2985)
release-please[bot] a8531fd
chore(main): release 6.63.1-SNAPSHOT (#2991)
release-please[bot] e74319c
chore: clean up some warnings and malformed comments (#2977)
olavloite 248744b
chore(deps): update dependency com.google.cloud:google-cloud-spanner …
renovate-bot 629fb32
feat: add endpoint connection URL property (#2969)
olavloite b72581c
feat: support max_commit_delay in Connection API (#2954)
olavloite d05695b
chore: minor improvements to default benchmarks. (#2993)
arpan14 417a83c
build(deps): update dependency org.jacoco:jacoco-maven-plugin to v0.8…
renovate-bot 2d6f4f3
chore: add regex to match unmanaged dependency check (#1941) (#2971)
gcf-owl-bot[bot] d2b4862
feat: Add SessionPoolOptions, SpannerOptions protos in executor proto…
gcf-owl-bot[bot] b8074b5
Merge branch 'main' into pg-oid
tlhquynh cdd1c91
Merge branch 'main' into pg-oid
tlhquynh b63740d
chore: Remove unused CLIRR entries
tlhquynh File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,3 +31,5 @@ docs/ | |
*.pyc | ||
|
||
.flattened-pom.xml | ||
.java-version | ||
.vscode/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should not be necessary to add the internal methods to this file, if those methods are given a default implementation that just throws UnsupportedOperationException
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These entries in
clirr-ignored-differences.xml
forStructReader
can be removed, as we don't have those methods anymore.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
final nit (I promise :-)): I think that the two remaining additions to this file for getPgOid() and getPgOidArray() can also be removed, as these methods are no longer added to the
Value
class.