Skip to content

Upgrade guava version to 33.4.8 #12219

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

zrlw
Copy link

@zrlw zrlw commented Jul 15, 2025

#12215
Guava seems to call a deprecated sun.misc.Unsafe::objectFieldOffset method which might be removed in a future JDK release.
There are still a few things to do for -android versions, which are tracked in google/guava#7742,
see details at google/guava#7811

Copy link

linux-foundation-easycla bot commented Jul 15, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@zrlw zrlw force-pushed the guava-33.4.8-jre branch from db60d82 to bf9cfba Compare July 15, 2025 02:16
@zrlw zrlw force-pushed the guava-33.4.8-jre branch from bf9cfba to 497c16c Compare July 15, 2025 02:40
@zrlw zrlw changed the title Upgrade guava version to 33.4.8-jre Upgrade guava version to 33.4.8 Jul 15, 2025
@shivaspeaks shivaspeaks added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Jul 15, 2025
@grpc-kokoro grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Jul 15, 2025
@zrlw
Copy link
Author

zrlw commented Jul 16, 2025

# error-prone 2.31.0+ blocked on https://github.com/grpc/grpc-java/issues/10152
# It breaks Bazel (ArrayIndexOutOfBoundsException in turbine) and Dexing ("D8:
# java.lang.NullPointerException"). We can trivially upgrade the Bazel CI to
# 6.3.0+ (https://github.com/bazelbuild/bazel/issues/18743).

According to https://issuetracker.google.com/issues/342522142#comment8 (Jun 6, 2024):

This was fixed in D8/R8 compilers a little more than a year ago. The issue tracking this on D8/R8 is b/281536562, but that is an internal bug.
The fix landed in 8.3.3-dev, and was part of AGP 8.3. It was backported to 8.1.44 and 8.0.46 for AGP 8.1 and 8.0.
It was not backported to D8/R8 4.0 for AGP 7.4. However, you should be able to use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed. To use a specific D8/R8 version merge the following into settings.gradle or settings.gradle.kts:

pluginManagement {
    buildscript {
        repositories {
            mavenCentral()
            maven {
                url = uri("https://storage.googleapis.com/r8-releases/raw")
            }
        }
        dependencies {
            classpath("com.android.tools:r8:8.1.44")
        }
    }
}

@zrlw
Copy link
Author

zrlw commented Jul 16, 2025

@shivaspeaks PTAL

@ejona86
Copy link
Member

ejona86 commented Jul 17, 2025

We're trying to get a release made (v1.74.0). Once it is made, we'll look at bumping the dependency versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants