[ https://issues.apache.org/jira/browse/CASSANDRA-3658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175203#comment-13175203 ]
Hudson commented on CASSANDRA-3658: ----------------------------------- Integrated in Cassandra #1266 (See [https://builds.apache.org/job/Cassandra/1266/]) Fix minor issues reported by FindBugs patch by slebresne; reviewed by jbellis for CASSANDRA-3658 slebresne : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1222476 Files : * /cassandra/trunk/CHANGES.txt * /cassandra/trunk/src/java/org/apache/cassandra/cli/CliClient.java * /cassandra/trunk/src/java/org/apache/cassandra/config/ReplicationStrategy.java * /cassandra/trunk/src/java/org/apache/cassandra/db/ArrayBackedSortedColumns.java * /cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamily.java * /cassandra/trunk/src/java/org/apache/cassandra/db/ExpiringColumn.java * /cassandra/trunk/src/java/org/apache/cassandra/db/RowIteratorFactory.java * /cassandra/trunk/src/java/org/apache/cassandra/db/compaction/SizeTieredCompactionStrategy.java * /cassandra/trunk/src/java/org/apache/cassandra/db/filter/QueryFilter.java * /cassandra/trunk/src/java/org/apache/cassandra/db/marshal/AbstractType.java * /cassandra/trunk/src/java/org/apache/cassandra/db/marshal/DynamicCompositeType.java * /cassandra/trunk/src/java/org/apache/cassandra/db/marshal/ReversedType.java * /cassandra/trunk/src/java/org/apache/cassandra/dht/LocalToken.java * /cassandra/trunk/src/java/org/apache/cassandra/dht/Token.java * /cassandra/trunk/src/java/org/apache/cassandra/io/compress/CompressionMetadata.java * /cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTableLoader.java * /cassandra/trunk/src/java/org/apache/cassandra/locator/PropertyFileSnitch.java * /cassandra/trunk/src/java/org/apache/cassandra/net/ProtocolHeader.java * /cassandra/trunk/src/java/org/apache/cassandra/service/AntiEntropyService.java * /cassandra/trunk/src/java/org/apache/cassandra/streaming/FileStreamTask.java * /cassandra/trunk/src/java/org/apache/cassandra/utils/EstimatedHistogram.java * /cassandra/trunk/src/java/org/apache/cassandra/utils/FBUtilities.java * /cassandra/trunk/src/java/org/apache/cassandra/utils/NodeId.java > Fix smallish problems find by FindBugs > -------------------------------------- > > Key: CASSANDRA-3658 > URL: https://issues.apache.org/jira/browse/CASSANDRA-3658 > Project: Cassandra > Issue Type: Bug > Components: Core > Reporter: Sylvain Lebresne > Assignee: Sylvain Lebresne > Priority: Minor > Labels: fingbugs > Fix For: 1.1 > > Attachments: 0001-Respect-Future-semantic.patch, > 0002-Avoid-race-when-reloading-snitch-file.patch, > 0003-use-static-inner-class-when-possible.patch, 0004-Remove-dead-code.patch, > 0005-Protect-against-signed-byte-extension.patch, > 0006-Add-hashCode-method-when-equals-is-overriden.patch, > 0007-Inverse-argument-of-compare-instead-of-negating-to-a.patch, > 0008-stop-pretending-Token-is-Serializable-LocalToken-is-.patch, > 0009-remove-useless-assert-that-is-always-true.patch, > 0010-Add-equals-and-hashCode-to-Expiring-column.patch > > > I've just run (the newly released) FindBugs 2 out of curiosity. Attaching a > number of patches related to issue raised by it. There is nothing major at > all so all patches are against trunk. > I've tried keep each issue to it's own patch with a self describing title. It > far from covers all FindBugs alerts, but it's a picky tool so I've tried to > address only what felt at least vaguely useful. Those are still mostly nits > (only patch 2 is probably an actual bug). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira