From 5cc7a11ab8684af2f9b127341f4029657b5fa3b2 Mon Sep 17 00:00:00 2001 From: Dave Brosius Date: Sat, 6 Sep 2014 14:39:54 -0400 Subject: [PATCH] fix boxing confusion --- src/java/org/apache/cassandra/db/Memtable.java | 2 +- src/java/org/apache/cassandra/dht/RangeStreamer.java | 2 +- .../org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java | 4 ++-- src/java/org/apache/cassandra/tools/StandaloneSplitter.java | 2 +- .../cassandra/stress/settings/OptionAnyProbabilities.java | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/java/org/apache/cassandra/db/Memtable.java b/src/java/org/apache/cassandra/db/Memtable.java index 4825676a86..ef157a38dd 100644 --- a/src/java/org/apache/cassandra/db/Memtable.java +++ b/src/java/org/apache/cassandra/db/Memtable.java @@ -52,7 +52,7 @@ public class Memtable private static final Logger logger = LoggerFactory.getLogger(Memtable.class); static final MemtablePool MEMORY_POOL = DatabaseDescriptor.getMemtableAllocatorPool(); - private static final int ROW_OVERHEAD_HEAP_SIZE = estimateRowOverhead(Integer.valueOf(System.getProperty("cassandra.memtable_row_overhead_computation_step", "100000"))); + private static final int ROW_OVERHEAD_HEAP_SIZE = estimateRowOverhead(Integer.parseInt(System.getProperty("cassandra.memtable_row_overhead_computation_step", "100000"))); private final MemtableAllocator allocator; private final AtomicLong liveDataSize = new AtomicLong(0); diff --git a/src/java/org/apache/cassandra/dht/RangeStreamer.java b/src/java/org/apache/cassandra/dht/RangeStreamer.java index d84a9519de..be58d77f7c 100644 --- a/src/java/org/apache/cassandra/dht/RangeStreamer.java +++ b/src/java/org/apache/cassandra/dht/RangeStreamer.java @@ -47,7 +47,7 @@ import org.apache.cassandra.utils.FBUtilities; public class RangeStreamer { private static final Logger logger = LoggerFactory.getLogger(RangeStreamer.class); - public static final boolean useStrictConsistency = Boolean.valueOf(System.getProperty("cassandra.consistent.rangemovement","true")); + public static final boolean useStrictConsistency = Boolean.parseBoolean(System.getProperty("cassandra.consistent.rangemovement","true")); private final Collection tokens; private final TokenMetadata metadata; private final InetAddress address; diff --git a/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java b/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java index 73a291b050..30d742c963 100644 --- a/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java +++ b/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java @@ -525,7 +525,7 @@ public class CqlConfigHelper String setting = conf.get(parameter); if (setting == null) return Optional.absent(); - return Optional.of(Integer.parseInt(setting)); + return Optional.of(Integer.valueOf(setting)); } private static Optional getBooleanSetting(String parameter, Configuration conf) @@ -533,7 +533,7 @@ public class CqlConfigHelper String setting = conf.get(parameter); if (setting == null) return Optional.absent(); - return Optional.of(Boolean.parseBoolean(setting)); + return Optional.of(Boolean.valueOf(setting)); } private static Optional getStringSetting(String parameter, Configuration conf) diff --git a/src/java/org/apache/cassandra/tools/StandaloneSplitter.java b/src/java/org/apache/cassandra/tools/StandaloneSplitter.java index bcf986916a..3baf833db5 100644 --- a/src/java/org/apache/cassandra/tools/StandaloneSplitter.java +++ b/src/java/org/apache/cassandra/tools/StandaloneSplitter.java @@ -201,7 +201,7 @@ public class StandaloneSplitter opts.sizeInMB = DEFAULT_SSTABLE_SIZE; if (cmd.hasOption(SIZE_OPTION)) - opts.sizeInMB = Integer.valueOf(cmd.getOptionValue(SIZE_OPTION)); + opts.sizeInMB = Integer.parseInt(cmd.getOptionValue(SIZE_OPTION)); return opts; } diff --git a/tools/stress/src/org/apache/cassandra/stress/settings/OptionAnyProbabilities.java b/tools/stress/src/org/apache/cassandra/stress/settings/OptionAnyProbabilities.java index b685294707..6f8e9035d0 100644 --- a/tools/stress/src/org/apache/cassandra/stress/settings/OptionAnyProbabilities.java +++ b/tools/stress/src/org/apache/cassandra/stress/settings/OptionAnyProbabilities.java @@ -48,7 +48,7 @@ public final class OptionAnyProbabilities extends OptionMulti String[] args = param.split("="); if (args.length == 2 && args[1].length() > 0 && args[0].length() > 0) { - if (options.put(args[0], Double.parseDouble(args[1])) != null) + if (options.put(args[0], Double.valueOf(args[1])) != null) throw new IllegalArgumentException(args[0] + " set twice"); return true; }