Merge branch 'cassandra-3.0' into cassandra-3.3

Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/628c0767
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/628c0767
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/628c0767

Branch: refs/heads/trunk
Commit: 628c07678c022c5f062cba760056d9f22bc244cb
Parents: 09364e0 a3ae9d8
Author: Yuki Morishita <[email protected]>
Authored: Thu Jan 14 09:58:43 2016 -0600
Committer: Yuki Morishita <[email protected]>
Committed: Thu Jan 14 09:58:43 2016 -0600

----------------------------------------------------------------------
 CHANGES.txt                                                      | 1 +
 .../org/apache/cassandra/db/compaction/CompactionManager.java    | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cassandra/blob/628c0767/CHANGES.txt
----------------------------------------------------------------------
diff --cc CHANGES.txt
index bf873fc,aebdc26..bfe1730
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@@ -9,9 -5,22 +9,10 @@@ Merged from 3.0
     tombstone (CASSANDRA-10743)
   * MV should use the maximum timestamp of the primary key (CASSANDRA-10910)
   * Fix potential assertion error during compaction (CASSANDRA-10944)
 - * Fix counting of received sstables in streaming (CASSANDRA-10949)
 - * Implement hints compression (CASSANDRA-9428)
 - * Fix potential assertion error when reading static columns (CASSANDRA-10903)
 - * Avoid NoSuchElementException when executing empty batch (CASSANDRA-10711)
 - * Avoid building PartitionUpdate in toString (CASSANDRA-10897)
 - * Reduce heap spent when receiving many SSTables (CASSANDRA-10797)
 - * Add back support for 3rd party auth providers to bulk loader 
(CASSANDRA-10873)
 - * Eliminate the dependency on jgrapht for UDT resolution (CASSANDRA-10653)
 - * (Hadoop) Close Clusters and Sessions in Hadoop Input/Output classes 
(CASSANDRA-10837)
 - * Fix sstableloader not working with upper case keyspace name 
(CASSANDRA-10806)
  Merged from 2.2:
+  * Avoid NPE when performing sstable tasks (scrub etc.) (CASSANDRA-10980)
   * Make sure client gets tombstone overwhelmed warning (CASSANDRA-9465)
   * Fix error streaming section more than 2GB (CASSANDRA-10961)
 - * (cqlsh) Also apply --connect-timeout to control connection
 -   timeout (CASSANDRA-10959)
   * Histogram buckets exposed in jmx are sorted incorrectly (CASSANDRA-10975)
   * Enable GC logging by default (CASSANDRA-10140)
   * Optimize pending range computation (CASSANDRA-9258)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/628c0767/src/java/org/apache/cassandra/db/compaction/CompactionManager.java
----------------------------------------------------------------------
diff --cc src/java/org/apache/cassandra/db/compaction/CompactionManager.java
index 1f39767,28e4174..3e7b198
--- a/src/java/org/apache/cassandra/db/compaction/CompactionManager.java
+++ b/src/java/org/apache/cassandra/db/compaction/CompactionManager.java
@@@ -266,12 -265,12 +266,12 @@@ public class CompactionManager implemen
      private AllSSTableOpStatus parallelAllSSTableOperation(final 
ColumnFamilyStore cfs, final OneSSTableOperation operation, OperationType 
operationType) throws ExecutionException, InterruptedException
      {
          List<LifecycleTransaction> transactions = new ArrayList<>();
-         try (LifecycleTransaction compacting = 
cfs.markAllCompacting(operationType);)
+         try (LifecycleTransaction compacting = 
cfs.markAllCompacting(operationType))
          {
-             Iterable<SSTableReader> sstables = 
Lists.newArrayList(operation.filterSSTables(compacting));
+             Iterable<SSTableReader> sstables = compacting != null ? 
Lists.newArrayList(operation.filterSSTables(compacting)) : 
Collections.<SSTableReader>emptyList();
              if (Iterables.isEmpty(sstables))
              {
 -                logger.info("No sstables for {}.{}", cfs.keyspace.getName(), 
cfs.name);
 +                logger.info("No sstables to {} for {}.{}", 
operationType.name(), cfs.keyspace.getName(), cfs.name);
                  return AllSSTableOpStatus.SUCCESSFUL;
              }
  

Reply via email to