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

2.2 eclipse-warnings

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Normal
    • Resolution: Fixed
    • Fix Version/s: 2.2.16
    • Component/s: Build
    • Labels:
      None

      Description

      Cassandra-2.2 artifact builds are failing from eclipse-warnings.

      # 11/11/19 2:58:41 PM UTC
      # Eclipse Compiler for Java(TM) v20150120-1634, 3.10.2, Copyright IBM Corp 2000, 2013. All rights reserved.
      incorrect classpath: /home/jenkins/jenkins-slave/workspace/Cassandra-2.2-artifacts/build/cobertura/classes
      ----------
      1. ERROR in /home/jenkins/jenkins-slave/workspace/Cassandra-2.2-artifacts/src/java/org/apache/cassandra/db/compaction/CompactionManager.java (at line 880)
      	ISSTableScanner scanner = cleanupStrategy.getScanner(sstable, getRateLimiter());
      	                ^^^^^^^
      Resource 'scanner' should be managed by try-with-resource
      ----------
      ----------
      2. ERROR in /home/jenkins/jenkins-slave/workspace/Cassandra-2.2-artifacts/src/java/org/apache/cassandra/db/compaction/LeveledCompactionStrategy.java (at line 257)
      	scanners.add(new LeveledScanner(intersecting, range));
      	             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      Potential resource leak: '<unassigned Closeable value>' may not be closed
      ----------
      ----------
      3. ERROR in /home/jenkins/jenkins-slave/workspace/Cassandra-2.2-artifacts/src/java/org/apache/cassandra/tools/SSTableExport.java (at line 315)
      	ISSTableScanner scanner = reader.getScanner();
      	                ^^^^^^^
      Resource 'scanner' should be managed by try-with-resource
      ----------
      3 problems (3 errors)
      

      ref: https://builds.apache.org/job/Cassandra-2.2-artifacts/180/artifact/build/ecj/eclipse_compiler_checks.txt

        Attachments

          Activity

            People

            • Assignee:
              mck Michael Semb Wever
              Reporter:
              mck Michael Semb Wever
              Authors:
              Michael Semb Wever
              Reviewers:
              Benedict Elliott Smith
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: