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

Add benchmark code for spanner subquery #587

Merged

Conversation

yfuruyama
Copy link
Contributor

This PR contains the Cloud Spanner's benchmark code for my upcoming medium blog (in Japanese). I will update README once the blog post is published.

@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines. label Dec 14, 2020
@google-cla google-cla bot added the cla: yes All committers have signed a CLA label Dec 14, 2020
@yfuruyama
Copy link
Contributor Author

@jaketf @sabhyankar
Could you take a look at this PR?

@jaketf jaketf self-requested a review December 15, 2020 17:22
Copy link

@jaketf jaketf left a comment

Choose a reason for hiding this comment

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

source looks good.
some notes on docs.
this could use some test coverage.

@yfuruyama yfuruyama requested a review from jaketf December 16, 2020 00:24
@yfuruyama
Copy link
Contributor Author

@jaketf Thank you so much for the detailed comment! I fixed the code and README accordingly. Could you please review it again?

@jaketf jaketf merged commit c6299d9 into GoogleCloudPlatform:main Dec 16, 2020
@yfuruyama yfuruyama deleted the example-spanner-interleave-subquery branch December 16, 2020 01:08
rosmo pushed a commit to rosmo/professional-services that referenced this pull request Feb 1, 2021
* Add benchmark code for spanner subquery

* Update toplevel README

* Apply suggestions from code review

Co-authored-by: Jacob Ferriero <jferriero@google.com>

* Fix README and benchmark pattern

* Add how to stop benchmark

Co-authored-by: Jacob Ferriero <jferriero@google.com>
rosmo pushed a commit to rosmo/professional-services that referenced this pull request Mar 17, 2022
* Add benchmark code for spanner subquery

* Update toplevel README

* Apply suggestions from code review

Co-authored-by: Jacob Ferriero <jferriero@google.com>

* Fix README and benchmark pattern

* Add how to stop benchmark

Co-authored-by: Jacob Ferriero <jferriero@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes All committers have signed a CLA size/L Denotes a PR that changes 100-499 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants