Skip to content

datastore: force import grpc-google-common-protos:0.1.9 - #2029

Merged
pongad merged 2 commits into
googleapis:masterfrom
pongad:datastore-versioning
May 4, 2017
Merged

datastore: force import grpc-google-common-protos:0.1.9#2029
pongad merged 2 commits into
googleapis:masterfrom
pongad:datastore-versioning

Conversation

@pongad

@pongad pongad commented May 3, 2017

Copy link
Copy Markdown
Contributor

Also removes grpc-all dependency from top-level pom.xml; it is unused.

Datastore is tested with mvn verify.
Depedency versions are shown to not conflict by
creating and compiling a test project that depends
only on google-cloud.
Since it does not inherit the version resolution
from google-cloud-java/pom.xml,
it more closely resembles user project.

Also removes grpc-all dependency from top-level pom.xml; it is unused.

Datastore is tested with `mvn verify`.
Depedency versions are shown to not conflict by
creating and compiling a test project that depends
only on google-cloud.
Since it does not inherit the version resolution
from google-cloud-java/pom.xml,
it more closely resembles user project.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 3, 2017
@coveralls

Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 1985918 on pongad:datastore-versioning into ** on GoogleCloudPlatform:master**.

@vam-google vam-google left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please ensure that this works as expected:
rm -r ~/.m2/repository
mvn clean install -DskipITs (from google-cloud-java repo root)

Comment thread google-cloud-datastore/pom.xml Outdated
<dependency>
<groupId>com.google.api.grpc</groupId>
<artifactId>grpc-google-common-protos</artifactId>
<version>0.1.9</version>

This comment was marked as spam.

This comment was marked as spam.

@pongad

pongad commented May 3, 2017

Copy link
Copy Markdown
Contributor Author

@vam-google PTAL

@coveralls

Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 6a6dc8a on pongad:datastore-versioning into ** on GoogleCloudPlatform:master**.

@vam-google vam-google left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

<groupId>com.google.api.grpc</groupId>
<artifactId>grpc-google-common-protos</artifactId>
<version>${generatedProto.version}</version>
</dependency>

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@pongad

pongad commented May 4, 2017

Copy link
Copy Markdown
Contributor Author

Merging with @vam-google 's +2. @garrettjonesgoogle I'll follow up in email.

@pongad
pongad merged commit 5d50249 into googleapis:master May 4, 2017
@pongad
pongad deleted the datastore-versioning branch May 4, 2017 04:06
chingor13 pushed a commit that referenced this pull request Feb 20, 2026
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
meltsufin pushed a commit that referenced this pull request Apr 29, 2026
meltsufin pushed a commit that referenced this pull request May 1, 2026
meltsufin pushed a commit that referenced this pull request May 2, 2026
…plugin to v3.1.4 (#2029)

Co-authored-by: Tom Andersen <tom-andersen@users.noreply.github.com>
benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026
…duces correct results (googleapis#2029)

Change-Id: I909ec95bebe87219d66b387cd80e7095e75252a2

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a [bug/issue](https://togithub.com/googleapis/java-bigtable/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)
- [ ] Rollback plan is reviewed and LGTMed

Fixes #<issue_number_goes_here> ☕️

If you write sample code, please follow the [samples format](
https://togithub.com/GoogleCloudPlatform/java-docs-samples/blob/main/SAMPLE_FORMAT.md).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants