----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15245/ -----------------------------------------------------------
(Updated Nov. 7, 2013, 9:18 p.m.) Review request for accumulo, Sean Busbey, Christopher Tubbs, John Vines, and kturner. Changes ------- Updated diff with suggested changes integrated. Repository: accumulo Description ------- (was https://reviews.apache.org/r/14972/ ; created new review created so I get responses instead of Vines) Rebased on 1.6.0-SNAPSHOT's current HEAD (which is pretty close to master), and feedback from the last review integrated. Still need to clean up all the references to newly deprecated ZKInstance constructors and Input/OutputFormat config setters, but wanted to get this up for feedback sooner. Diffs (updated) ----- core/src/main/java/org/apache/accumulo/core/Constants.java 9db0c40 core/src/main/java/org/apache/accumulo/core/cli/ClientOpts.java 9247d56 core/src/main/java/org/apache/accumulo/core/client/Instance.java 95fc933 core/src/main/java/org/apache/accumulo/core/client/ZooKeeperInstance.java a85b72e core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java 6b2a1cf core/src/main/java/org/apache/accumulo/core/client/impl/MasterClient.java 32c80f9 core/src/main/java/org/apache/accumulo/core/client/impl/ServerClient.java 218bd36 core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchReaderIterator.java 0376304 core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java 0dd86bf core/src/main/java/org/apache/accumulo/core/client/impl/ThriftTransportKey.java f07139d core/src/main/java/org/apache/accumulo/core/client/impl/ThriftTransportPool.java e7dabb5 core/src/main/java/org/apache/accumulo/core/client/mapred/AbstractInputFormat.java 856936e core/src/main/java/org/apache/accumulo/core/client/mapred/AccumuloInputFormat.java cccd7b8 core/src/main/java/org/apache/accumulo/core/client/mapred/AccumuloMultiTableInputFormat.java 61838db core/src/main/java/org/apache/accumulo/core/client/mapred/AccumuloOutputFormat.java 908b8b3 core/src/main/java/org/apache/accumulo/core/client/mapred/AccumuloRowInputFormat.java fe5003b core/src/main/java/org/apache/accumulo/core/client/mapreduce/AbstractInputFormat.java 626a785 core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloInputFormat.java 9ecae53 core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java 727bfec core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloRowInputFormat.java 992990d core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java 4f8cdb6 core/src/main/java/org/apache/accumulo/core/client/mock/MockInstance.java d391464 core/src/main/java/org/apache/accumulo/core/conf/AccumuloConfiguration.java 0a456ba core/src/main/java/org/apache/accumulo/core/conf/ClientConfiguration.java PRE-CREATION core/src/main/java/org/apache/accumulo/core/conf/Property.java b915ee4 core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java fc45442 core/src/main/java/org/apache/accumulo/core/security/Credentials.java 0552e7e core/src/main/java/org/apache/accumulo/core/security/SecurityUtil.java 8add1a7 core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java PRE-CREATION core/src/main/java/org/apache/accumulo/core/util/ThriftUtil.java e8dd6a2 core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java 52e1d04 core/src/main/java/org/apache/accumulo/core/util/shell/ShellOptionsJC.java cb1f1c8 core/src/test/java/org/apache/accumulo/core/client/impl/TabletLocatorImplTest.java 8f3fa1d core/src/test/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBaseTest.java 62564fa core/src/test/java/org/apache/accumulo/core/conf/ClientConfigurationTest.java PRE-CREATION core/src/test/java/org/apache/accumulo/core/util/shell/ShellSetInstanceTest.java 23ca13a examples/simple/src/main/java/org/apache/accumulo/examples/simple/filedata/FileDataQuery.java ecb42c7 examples/simple/src/main/java/org/apache/accumulo/examples/simple/mapreduce/TokenFileWordCount.java 674d793 minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloCluster.java 42882bb minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java bfa7922 minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloInstance.java 540d7ae minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterGCTest.java 6470ce1 minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterTest.java 26a1546 proxy/src/main/java/org/apache/accumulo/proxy/ProxyServer.java c92f73b server/base/src/main/java/org/apache/accumulo/server/cli/ClientOnDefaultTable.java 53f5ac2 server/base/src/main/java/org/apache/accumulo/server/cli/ClientOnRequiredTable.java e9e9bf1 server/base/src/main/java/org/apache/accumulo/server/cli/ClientOpts.java 6f3516a server/base/src/main/java/org/apache/accumulo/server/client/BulkImporter.java 606941d server/base/src/main/java/org/apache/accumulo/server/client/HdfsZooInstance.java 9e6bbe7 server/base/src/main/java/org/apache/accumulo/server/util/TServerUtils.java 8abd104 test/pom.xml 8e4c152 test/src/main/java/org/apache/accumulo/test/IMMLGBenchmark.java 948a741 test/src/main/java/org/apache/accumulo/test/TestIngest.java 972a20e test/src/main/java/org/apache/accumulo/test/performance/metadata/MetadataBatchScanTest.java c6ad74f test/src/main/java/org/apache/accumulo/test/performance/thrift/NullTserver.java 78b2564 test/src/main/java/org/apache/accumulo/test/randomwalk/State.java ea6d8ed test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java e104c99 test/src/main/java/org/apache/accumulo/test/randomwalk/sequential/MapRedVerifyTool.java 6c7cc63 test/src/main/java/org/apache/accumulo/test/scalability/ScaleTest.java e6ba77b test/src/test/java/org/apache/accumulo/test/ShellServerIT.java 31867f3 test/src/test/java/org/apache/accumulo/test/functional/AbstractMacIT.java d24b85b test/src/test/java/org/apache/accumulo/test/functional/AccumuloInputFormatIT.java d38536a test/src/test/java/org/apache/accumulo/test/functional/BulkIT.java 2fb5827 test/src/test/java/org/apache/accumulo/test/functional/ConcurrencyIT.java c3d3160 test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java 3f60f1d test/src/test/java/org/apache/accumulo/test/functional/MapReduceIT.java 9e42e55 test/src/test/java/org/apache/accumulo/test/functional/ScannerIT.java 7913089 test/src/test/java/org/apache/accumulo/test/functional/ShutdownIT.java 8d58821 test/src/test/java/org/apache/accumulo/test/functional/SimpleMacIT.java 9086f13 test/src/test/java/org/apache/accumulo/test/functional/SslIT.java PRE-CREATION test/src/test/java/org/apache/accumulo/test/functional/SslWithClientAuthIT.java PRE-CREATION test/src/test/java/org/apache/accumulo/test/util/CertUtils.java PRE-CREATION test/src/test/java/org/apache/accumulo/test/util/CertUtilsTest.java PRE-CREATION Diff: https://reviews.apache.org/r/15245/diff/ Testing ------- Thanks, Michael Berman
