Skip to content

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

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

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

Conversation

@pongad
Copy link
Copy Markdown
Contributor

@pongad pongad commented May 3, 2017

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**.

Copy link
Copy Markdown
Contributor

@vam-google vam-google left a comment

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
Copy link
Copy Markdown
Contributor Author

pongad commented May 3, 2017

@vam-google PTAL

@coveralls
Copy link
Copy Markdown

Coverage Status

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

Copy link
Copy Markdown
Contributor

@vam-google vam-google left a comment

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
Copy link
Copy Markdown
Contributor Author

pongad commented May 4, 2017

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
🤖 I have created a release *beep* *boop*
---


## [2.34.1](https://togithub.com/googleapis/java-bigquerystorage/compare/v2.34.0...v2.34.1) (2023-03-21)


### Bug Fixes

* Add service_yaml_parameters to `java_gapic_library` targets ([#2034](https://togithub.com/googleapis/java-bigquerystorage/issues/2034)) ([8dae87c](https://togithub.com/googleapis/java-bigquerystorage/commit/8dae87cff06c7d08b9d597be9c66de570739abf4))


### Dependencies

* Update dependency com.google.cloud:google-cloud-bigquery to v2.23.2 ([#2029](https://togithub.com/googleapis/java-bigquerystorage/issues/2029)) ([e9c0152](https://togithub.com/googleapis/java-bigquerystorage/commit/e9c0152c2b098d7fb2d1de4d535d636a3ac9f90e))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.5.0 ([#2050](https://togithub.com/googleapis/java-bigquerystorage/issues/2050)) ([349092f](https://togithub.com/googleapis/java-bigquerystorage/commit/349092f28f6017b1df5b325e9a9036c311e70546))
* Update dependency com.google.http-client:google-http-client to v1.43.1 ([#2038](https://togithub.com/googleapis/java-bigquerystorage/issues/2038)) ([5e865c9](https://togithub.com/googleapis/java-bigquerystorage/commit/5e865c9b3779f0d20080eead241f1b5858156880))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
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 May 2, 2026
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