Uploaded image for project: 'Apache Cassandra'
  1. Apache Cassandra
  2. CASSANDRA-18645

Upgrade guava on trunk

    XMLWordPrintableJSON

Details

    • Operability
    • Normal
    • All
    • None
    • Hide

      We must remove the exclusion of failureaccess - Guava InternalFutureFailureAccess and InternalFuturesContains.
      From maven:
      "com.google.common.util.concurrent.internal.InternalFutureFailureAccess and InternalFutures. Most Guava users will never need to use this artifact. Its classes are conceptually a part of Guava, but they were moved to a separate artifact so that Android libraries can use them without pulling in all of Guava (just as they can use ListenableFuture by depending on the listenablefuture artifact)."

      I went carefully through the changelog. The only thing that came out is that two security vulnerabilities have been fixed in the latest versions, so we no longer need the suppressions. I removed those.
      Patch, CI: J8, J11, J17

      Show
      We must remove the exclusion of failureaccess - Guava InternalFutureFailureAccess and InternalFuturesContains. From maven: "com.google.common.util.concurrent.internal.InternalFutureFailureAccess and InternalFutures. Most Guava users will never need to use this artifact. Its classes are conceptually a part of Guava, but they were moved to a separate artifact so that Android libraries can use them without pulling in all of Guava (just as they can use ListenableFuture by depending on the listenablefuture artifact)." I went carefully through the changelog. The only thing that came out is that two security vulnerabilities have been fixed in the latest versions, so we no longer need the suppressions. I removed those. Patch, CI: J8, J11, J17

    Description

      Recently guava added JDK17 in CI and fixed some bugs down the road.

      Upgrading before the major 5.0 release is something we should do. 

      Also, the current version that Cassandra uses is from 2018. 

      Attachments

        Activity

          People

            e.dimitrova Ekaterina Dimitrova
            e.dimitrova Ekaterina Dimitrova
            Ekaterina Dimitrova
            Berenguer Blasi
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: