Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

we aren't using datanucleus anyway #1183

Merged
merged 3 commits into from
Oct 30, 2018
Merged

we aren't using datanucleus anyway #1183

merged 3 commits into from
Oct 30, 2018

Conversation

elharo
Copy link
Contributor

@elharo elharo commented Sep 21, 2018

@chingor13 Fix #1167

Not sure how to regenerate google-api-client-assembly/dependencies/google-http-client-jdo-dependencies.html (and why are we checking generated code into the repo?)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 21, 2018
@chingor13 chingor13 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 24, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 24, 2018
@chingor13 chingor13 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2018
@sduskis sduskis self-requested a review October 29, 2018 16:33
@sduskis sduskis requested a review from a team as a code owner October 29, 2018 16:35
@elharo
Copy link
Contributor Author

elharo commented Oct 29, 2018

Please merge this @sduskis

I do not have write access to this repo.

@chingor13
Copy link
Collaborator

I was holding off because I don't know the implications of removing datanucleus. It rewrites to bytecode to provide the JDO interface, so I'm not sure if we can remove it (or perhaps we no longer want to provide the jdo artifact?).

chingor13
chingor13 approved these changes Oct 30, 2018
View reviewed changes
Copy link
Collaborator

@chingor13 chingor13 left a comment

Choose a reason for hiding this comment

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

This one looks fine as we aren't using any of DataNucleus' features in the repo.

@chingor13 chingor13 merged commit 13407f9 into googleapis:master Oct 30, 2018
@chingor13 chingor13 mentioned this pull request Nov 9, 2018
Merged
sduskis pushed a commit that referenced this pull request Mar 12, 2019
@Bhanditz @sduskis
Google api java client (#1244)
c7afd0c 
* Fix Copyright notices (#1174)

* add OSGI metadata

* more default behavior

* add OSGI info

* whitespace

* Remove (c) from copyright notices

* Remove obsolete mercurial and google code detritus (#1175)

* add OSGI metadata

* more default behavior

* add OSGI info

* whitespace

* Remove out of date docs we don't use anymore now that we're on Github and git

* update appengine SDK (#1178)

* update minimum maven (#1179)

* Fix AppEngine tests (#1184)

* Add continuous integration Kokoro CI config and badges (#1171)

* Add continuous integration Kokoro CI config and badges

* Fix indentation and https

* Add CODEOWNERS and issue/pr templates (#1185)

*  App Engine standard uses Java 7 or Java 8, not 6 and not 9 or later (#1186)

* App Engine uses Java 7 or later

* App Engine uses Java 8 or earlier

* Add the ending Java 6 support notice to README (#1190)

* Add Kokoro release scripts (#1188)

* Add Kokoro release scripts

* Remove unnecessary extra maven-jar compile after upgrading plugins

* Add the appengine api

* skip tests

* Install play services

* Install play services only if not installed

* Install other dependencies

* Fix the BUGFIX_VERSION constant

* Fix CODEOWNERS format

* Annotate versions for replacement (#1192)

* Release v1.26.0 (#1193)

* Release v1.26.0

* Actually use the nexus-staging-maven-plugin

* Fix leftover snapshot versions in assembly

* Fix snapshot version in assembly

* Bump next snapshot (#1194)

* Bump next snapshot

* Fix version replace region tag.

* Release improvements (#1195)

* Can run mvn assembly:single outside of build

* Filter dependency html files

* Use assembly to rename versioned properties files

* Remove datanucleus dependency (#1183)

* Fix possible NPE when missing os.version system property (#1210)

* Add tests for missing os.version system property

* Only append the OS name/version if we can get them from the system properties

* Fix test allow testing different system properties

* Fix tests

* Fix the reset of system properties

* Remove unused import

* undo change to AsesrtHeaderTransport

* Fix codestyle

* Delay request initialization for resumable upload (#1211)

* Delay request initialization for resumable upload until the content chunk is ready

* Fix javadoc link

* Adding test for a slow writer

* Make ContentChunk class static for internal clarity lint.

* Add final to ContentChunk fields set only in constructor

* Add google-api-client-bom artifact (#1213)

* Add google-api-client-bom artifact

* Apply README.md suggestions

* Release google-api-java-client v1.27.0 (#1214)

* Release v1.27.0

* bom needs developers section and release config/profile

* Bump next snapshot (#1215)

* Update guava to 26.0-android (#1218)

* Compile to Java 1.7 and fix javadoc links (#1222)

* Add Java 11 test configs (#1223)

* Add Java 11 test configs

* Update README badges

* Update maven-javadoc-plugin for Java 11

* Fix X-Goog-Api-Client java version parsing for Java 11

* Update README: edited and removed some redundancy (#1226)

* Fix old links to code.google.com  (#1225)

* Update google-api-client-dependencies.html

* updated links

fixed links

* Update package-info.java

* corrected links

* edited links

* Update google-api-client-appengine-dependencies.html

* Update google-api-client-appengine-dependencies.html

* Update google-http-client-appengine-dependencies.html

* Update google-oauth-client-appengine-dependencies.html

* Update GoogleAuthorizationCodeTokenRequest.java

* Update package-info.java

* Update GoogleOAuthConstants.java

* Update package-info.java

* Update package-info.java

* Update GoogleJsonError.java

* Remove usage of deprecated Backoff from google-http-java-client (#1221)

* Remove ClientLogin (#1224)

* Remove ClientLogin

* Bump clirr comparison version and note that we are removing beta classes

* Set expires_in to an hour instead of 1000 hours (#1229)

* Use maven enforcer plugin for maven version requirements (#1228)

* Handle the legacy endpoint in the MockTokenServerTransport (#1232)

* Handle the legacy endpoint in the MockTokenServerTransport with a warning

* Bump maven-surefire-plugin version for test environment fixes

* Release google-api-java-client v1.28.0 (#1233)

* Release v1.28.0

* Fix dependency for google-http-client-apache and Apache transport

* Fix promote/drop kokoro configs

* Bump next snapshot (#1234)

* 3929: Changed to Guava ByteStreams.copy() that has a bit better performance (#1239)

* Add automatic module name (#1235)

* 956: Chech for null to prevent autoboxing NPE. (#1241)

* 956: Chech for null to prevent autoboxing NPE.

* 956: Add an unit test.

* Infinite cycle with MediaHttpDownloader setContentRange download (#1242)

* 1101: Fixed infinite cycle with MediaHttpDownloader setContentRange download.

* 1101: Added unit tests.

* 1101: Fixed comments.

* Fix Replaced invalidateToken method to clearToken (#1243)

* Replaced invalidateToken method to clearToken

* Fix throw exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sduskis sduskis sduskis approved these changes

@chingor13 chingor13 chingor13 approved these changes

Assignees
No one assigned
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@elharo @chingor13 @sduskis @googlebot @kokoro-team