Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

feat: relocate GoogleJsonClientFeature from java-core to gax #1655

Merged
merged 3 commits into from
Apr 8, 2022

Conversation

mpeddada1
Copy link
Contributor

@mpeddada1 mpeddada1 commented Apr 8, 2022

https://github.com/googleapis/google-http-java-client still uses Java 7 as the compiler target which is not compatible with the Feature interface so moving this to gax for now. See: https://www.graalvm.org/release-notes/19_0/ . Tested this out locally with Storage and Pub/Sub.
TODO:

  • Remove GoogleJsonClientFeature from java-core once this PR is merged.

@mpeddada1 mpeddada1 requested review from a team as code owners April 8, 2022 13:36
Copy link
Member

@meltsufin meltsufin left a comment

Choose a reason for hiding this comment

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

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Apr 8, 2022

We'll run into the same problem as #1651. Unfortunately, the call to access.registerSubtypeReachabilityHandler(BiConsumer<DuringAnalysisAccess, Class<?>> callback) in GoogleJsonClientFeature.loadApiClient() is incompatible with Java 7.

@meltsufin
Copy link
Member

Oh it's that one with Java 7 issue! Right!

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 8, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@mpeddada1 mpeddada1 merged commit 6e75ec7 into main Apr 8, 2022
@mpeddada1 mpeddada1 deleted the relocate-googlejson branch April 8, 2022 18:44
gcf-merge-on-green bot pushed a commit that referenced this pull request Apr 14, 2022
🤖 I have created a release *beep* *boop*
---


## [2.16.0](v2.15.0...v2.16.0) (2022-04-14)


### Features

* relocate ApiClientSubstitutions from java-core to gax ([#1651](#1651)) ([321103f](321103f))
* relocate GoogleJsonClientFeature from java-core to gax ([#1655](#1655)) ([6e75ec7](6e75ec7))
* relocate netty and gax substitutions from java-core ([#1650](#1650)) ([afd1917](afd1917))
* relocate Opencensus configurations from java-core to gax ([#1646](#1646)) ([b34ffb1](b34ffb1))
* relocate resource-config from java-core to gax ([#1656](#1656)) ([a9e5483](a9e5483))


### Dependencies

* google-common-protos 2.8.3 ([#1657](#1657)) ([24b9138](24b9138))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
suztomo pushed a commit to googleapis/sdk-platform-java that referenced this pull request Dec 16, 2022
🤖 I have created a release *beep* *boop*
---


## [2.16.0](googleapis/gax-java@v2.15.0...v2.16.0) (2022-04-14)


### Features

* relocate ApiClientSubstitutions from java-core to gax ([#1651](googleapis/gax-java#1651)) ([0d28727](googleapis/gax-java@0d28727))
* relocate GoogleJsonClientFeature from java-core to gax ([#1655](googleapis/gax-java#1655)) ([6f1ebe8](googleapis/gax-java@6f1ebe8))
* relocate netty and gax substitutions from java-core ([#1650](googleapis/gax-java#1650)) ([07a89cc](googleapis/gax-java@07a89cc))
* relocate Opencensus configurations from java-core to gax ([#1646](googleapis/gax-java#1646)) ([80e8b3f](googleapis/gax-java@80e8b3f))
* relocate resource-config from java-core to gax ([#1656](googleapis/gax-java#1656)) ([ec3f030](googleapis/gax-java@ec3f030))


### Dependencies

* google-common-protos 2.8.3 ([#1657](googleapis/gax-java#1657)) ([f9277bc](googleapis/gax-java@f9277bc))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants