You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by sl...@apache.org on 2013/09/17 15:32:53 UTC
[1/2] git commit: Fix potential AssertionError during tracing
Updated Branches:
refs/heads/cassandra-2.0 36f217819 -> 56b5c6f30
Fix potential AssertionError during tracing
patch by slebresne; reviewed by iamaleksey for CASSANDRA-6041
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/765d1e97
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/765d1e97
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/765d1e97
Branch: refs/heads/cassandra-2.0
Commit: 765d1e97ff7dbe50ce13297993720147ddb6eca0
Parents: 36f2178
Author: Sylvain Lebresne <sy...@datastax.com>
Authored: Tue Sep 17 15:29:18 2013 +0200
Committer: Sylvain Lebresne <sy...@datastax.com>
Committed: Tue Sep 17 15:30:15 2013 +0200
----------------------------------------------------------------------
CHANGES.txt | 1 +
src/java/org/apache/cassandra/tracing/Tracing.java | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/765d1e97/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 9f2b786..a9a658e 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -17,6 +17,7 @@
(CASSANDRA-6005)
* Fix paxos bugs (CASSANDRA-6012, 6013, 6023)
* Fix paged ranges with multiple replicas (CASSANDRA-6004)
+ * Fix potential AssertionError during tracing (CASSANDRA-6041)
Merged from 1.2:
1.2.10
* Fix possible divide-by-zero in HHOM (CASSANDRA-5990)
http://git-wip-us.apache.org/repos/asf/cassandra/blob/765d1e97/src/java/org/apache/cassandra/tracing/Tracing.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/tracing/Tracing.java b/src/java/org/apache/cassandra/tracing/Tracing.java
index 22b0244..f355c19 100644
--- a/src/java/org/apache/cassandra/tracing/Tracing.java
+++ b/src/java/org/apache/cassandra/tracing/Tracing.java
@@ -205,7 +205,7 @@ public class Tracing
public void runMayThrow() throws Exception
{
CFMetaData cfMeta = CFMetaData.TraceSessionsCf;
- ColumnFamily cf = ArrayBackedSortedColumns.factory.create(cfMeta);
+ ColumnFamily cf = TreeMapBackedSortedColumns.factory.create(cfMeta);
addColumn(cf, buildName(cfMeta, bytes("coordinator")), FBUtilities.getBroadcastAddress());
addParameterColumns(cf, parameters);
addColumn(cf, buildName(cfMeta, bytes("request")), request);
[2/2] git commit: Fix NPE in sstablesplit
Posted by sl...@apache.org.
Fix NPE in sstablesplit
patch by slebresne; reviewed by iamaleksey for CASSANDRA-6027
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/56b5c6f3
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/56b5c6f3
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/56b5c6f3
Branch: refs/heads/cassandra-2.0
Commit: 56b5c6f3024625b90f7a1deef1def7a7002b6e28
Parents: 765d1e9
Author: Sylvain Lebresne <sy...@datastax.com>
Authored: Tue Sep 17 15:31:13 2013 +0200
Committer: Sylvain Lebresne <sy...@datastax.com>
Committed: Tue Sep 17 15:31:13 2013 +0200
----------------------------------------------------------------------
CHANGES.txt | 1 +
.../org/apache/cassandra/db/compaction/CompactionController.java | 4 ++++
2 files changed, 5 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/56b5c6f3/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index a9a658e..f48fb5c 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -18,6 +18,7 @@
* Fix paxos bugs (CASSANDRA-6012, 6013, 6023)
* Fix paged ranges with multiple replicas (CASSANDRA-6004)
* Fix potential AssertionError during tracing (CASSANDRA-6041)
+ * Fix NPE in sstablesplit (CASSANDRA-6027)
Merged from 1.2:
1.2.10
* Fix possible divide-by-zero in HHOM (CASSANDRA-5990)
http://git-wip-us.apache.org/repos/asf/cassandra/blob/56b5c6f3/src/java/org/apache/cassandra/db/compaction/CompactionController.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/db/compaction/CompactionController.java b/src/java/org/apache/cassandra/db/compaction/CompactionController.java
index 3219926..9552895 100644
--- a/src/java/org/apache/cassandra/db/compaction/CompactionController.java
+++ b/src/java/org/apache/cassandra/db/compaction/CompactionController.java
@@ -100,6 +100,10 @@ public class CompactionController
public static Set<SSTableReader> getFullyExpiredSSTables(ColumnFamilyStore cfStore, Set<SSTableReader> compacting, Set<SSTableReader> overlapping, int gcBefore)
{
logger.debug("Checking droppable sstables in {}", cfStore);
+
+ if (compacting == null)
+ return Collections.<SSTableReader>emptySet();
+
List<SSTableReader> candidates = new ArrayList<SSTableReader>();
long minTimestamp = Integer.MAX_VALUE;