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

Deprecate Throwables.propagate usage

    XMLWordPrintableJSON

Details

    Description

      Google decided to deprecate guava Throwables.propagate method:

      We have 35 occurences in the trunk:

      $ rg -c 'Throwables.propagate' *
      src/java/org/apache/cassandra/streaming/StreamReader.java:1
      src/java/org/apache/cassandra/streaming/StreamTransferTask.java:1
      src/java/org/apache/cassandra/db/SnapshotDetailsTabularData.java:1
      src/java/org/apache/cassandra/db/Memtable.java:1
      src/java/org/apache/cassandra/db/ColumnFamilyStore.java:4
      src/java/org/apache/cassandra/cache/ChunkCache.java:2
      src/java/org/apache/cassandra/utils/WrappedRunnable.java:1
      src/java/org/apache/cassandra/hints/Hint.java:1
      src/java/org/apache/cassandra/tools/LoaderOptions.java:1
      src/java/org/apache/cassandra/tools/SSTableOfflineRelevel.java:1
      src/java/org/apache/cassandra/streaming/management/ProgressInfoCompositeData.java:3
      src/java/org/apache/cassandra/streaming/management/StreamStateCompositeData.java:2
      src/java/org/apache/cassandra/streaming/management/StreamSummaryCompositeData.java:2
      src/java/org/apache/cassandra/streaming/compress/CompressedStreamReader.java:1
      src/java/org/apache/cassandra/db/compaction/Scrubber.java:1
      src/java/org/apache/cassandra/db/compaction/Verifier.java:1
      src/java/org/apache/cassandra/db/compaction/CompactionHistoryTabularData.java:1
      src/java/org/apache/cassandra/db/compaction/Upgrader.java:1
      src/java/org/apache/cassandra/io/compress/CompressionMetadata.java:1
      src/java/org/apache/cassandra/streaming/management/SessionCompleteEventCompositeData.java:2
      src/java/org/apache/cassandra/io/sstable/SSTableSimpleWriter.java:1
      src/java/org/apache/cassandra/io/sstable/ISSTableScanner.java:1
      src/java/org/apache/cassandra/streaming/management/SessionInfoCompositeData.java:3
      src/java/org/apache/cassandra/io/sstable/SSTableSimpleUnsortedWriter.java:1
      

      I don't know if we want to remove all usages but we should at least check author's intention for each usage and refactor if needed.

      Attachments

        1. 14218-trunk.txt
          23 kB
          Kirk True
        2. 14218-trunk.txt
          22 kB
          Kirk True
        3. CASSANDRA-14218.txt
          32 kB
          Claude Warren

        Issue Links

          Activity

            People

              claude Claude Warren
              rha Romain Hardouin
              Claude Warren
              Caleb Rackliffe, David Capwell
              Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 1h 20m
                  1h 20m