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.
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
build, examples, README.md: Update protobuf gradle plugin version to 0.8.13 #7355
build, examples, README.md: Update protobuf gradle plugin version to 0.8.13 #7355
Conversation
…otobuf_gradle_plugin_version_to_0_8_13
Looks the Android build encounters some problem. I will look into it. |
2ef1a1f
to
283ad2b
Compare
…otobuf_gradle_plugin_version_to_0_8_13
b00f04c
to
e9021f0
Compare
There might be significant memory leaks, even 1 GB Metaspace is not enough. Trying 2 GB. |
dc4a687
to
ff534be
Compare
ff534be
to
c6fa4ff
Compare
Now it is working fine, with Ideally we should not use daemon in production. But building without daemon takes extremely long, which eventually timeout the job. /cc @ejona86 |
@@ -91,6 +91,7 @@ new_apk_size="$(stat --printf=%s $HELLO_WORLD_OUTPUT_DIR/apk/release/app-release | |||
cd $BASE_DIR/github/grpc-java | |||
git checkout HEAD^ | |||
./gradlew clean | |||
./gradlew --stop # use a new daemon to build the previous commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should go above the ./gradlew clean
since that clean is running at the new commit, or the ./gradlew clean
should be moved before the git checkout HEAD^
(preferable).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this. I know it was quite painful.
…0.8.13 (grpc#7355) Updated protobuf gradle plugin version to 0.8.13. Fixed Android Kokoro's memory issue by forcing to use a new Gradle daemon for building the previous commit.
No description provided.