Skip to content
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(spanner/spannertest): support LEFT OUTER JOIN with ON clause #2945

Merged
merged 1 commit into from Oct 1, 2020
Merged

feat(spanner/spannertest): support LEFT OUTER JOIN with ON clause #2945

merged 1 commit into from Oct 1, 2020

Conversation

dsymonds
Copy link
Contributor

@dsymonds dsymonds commented Oct 1, 2020

This lays the groundwork for supporting joins, and this initial subset
is a very common form.

Fixes #2462.

@dsymonds dsymonds added the api: spanner Issues related to the Spanner API. label Oct 1, 2020
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 1, 2020
@dsymonds
Copy link
Contributor Author

dsymonds commented Oct 1, 2020

/cc @adg

spanner/spannertest/db_query.go Show resolved Hide resolved
@dsymonds dsymonds added the automerge Merge the pull request once unit tests and other checks pass. label Oct 1, 2020
@gcf-merge-on-green
Copy link
Contributor

Your PR has attempted to merge for 3 hours. Please check that all required checks have passed, you have an automerge label, and that all your reviewers have approved the PR

@gcf-merge-on-green
Copy link
Contributor

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, or one of your required reviews was not approved. 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.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Oct 1, 2020
This lays the groundwork for supporting joins, and this initial subset
is a very common form.

Fixes #2462.
@dsymonds dsymonds added the automerge Merge the pull request once unit tests and other checks pass. label Oct 1, 2020
@dsymonds dsymonds merged commit a3af1e4 into googleapis:master Oct 1, 2020
@dsymonds dsymonds deleted the spannertest-join branch October 1, 2020 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the Spanner API. automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

spannertest: Support for (HASH) JOIN in select statement
2 participants