java-docs-samples icon indicating copy to clipboard operation
java-docs-samples copied to clipboard

docs: add samples for using timeouts

Open olavloite opened this issue 2 years ago • 7 comments

Adds samples for running queries and updates with timeouts. This is one of the most frequently asked questions for the Spring Data Spanner project.

olavloite avatar Jun 13 '23 12:06 olavloite

We also expect tests for each sample

averikitsch avatar Jun 13 '23 18:06 averikitsch

Here is the summary of changes.

You are about to add 2 region tags.

This comment is generated by snippet-bot. If you find problems with this result, please file an issue at: https://github.com/googleapis/repo-automation-bots/issues. To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • [ ] Refresh this comment

snippet-bot[bot] avatar Jun 14 '23 13:06 snippet-bot[bot]

We also expect tests for each sample

There are no tests or test infrastructure at all set up for this project, which makes adding tests for these samples difficult.

olavloite avatar Jun 14 '23 13:06 olavloite

We also expect tests for each sample

There are no tests or test infrastructure at all set up for this project, which makes adding tests for these samples difficult.

It is a part of our policy for samples to have tests: https://googlecloudplatform.github.io/samples-style-guide/#testing. Without tests samples are not maintainable. The api-spanner-java team owns the spanner samples, therefore without explicit consent from this owner we may not be able to accept these samples.

averikitsch avatar Jun 22 '23 16:06 averikitsch

api-spanner-java

Specifically the Spring Data Spanner project is maintained by a different team.

@meltsufin Could you chime in here, please? Do you know whether there's a specific reason that there are no tests set up for these samples? Or are there any plans for doing so?

olavloite avatar Jun 23 '23 05:06 olavloite

api-spanner-java

Specifically the Spring Data Spanner project is maintained by a different team.

@meltsufin Could you chime in here, please? Do you know whether there's a specific reason that there are no tests set up for these samples? Or are there any plans for doing so?

We have tests for Spring Data Spanner samples the we maintain in spring-cloud-gcp. We do not maintain java-docs-samples repo, however.

meltsufin avatar Sep 26 '23 16:09 meltsufin

@anguillanneuf Would you mind merging this PR? I don't think I have permission to merge anything in this repo.

olavloite avatar Oct 06 '23 17:10 olavloite