Return-Path: X-Original-To: apmail-accumulo-dev-archive@www.apache.org Delivered-To: apmail-accumulo-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 7F6CC1093D for ; Tue, 5 Nov 2013 19:45:01 +0000 (UTC) Received: (qmail 2987 invoked by uid 500); 5 Nov 2013 19:45:01 -0000 Delivered-To: apmail-accumulo-dev-archive@accumulo.apache.org Received: (qmail 2905 invoked by uid 500); 5 Nov 2013 19:45:01 -0000 Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list dev@accumulo.apache.org Received: (qmail 2895 invoked by uid 500); 5 Nov 2013 19:45:01 -0000 Delivered-To: apmail-incubator-accumulo-dev@incubator.apache.org Received: (qmail 2885 invoked by uid 99); 5 Nov 2013 19:45:01 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Nov 2013 19:45:01 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 62DDF1D3899; Tue, 5 Nov 2013 19:44:58 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4605459605209041043==" MIME-Version: 1.0 Subject: Re: Review Request 14972: ACCUMULO-1009 - add use of SSL for thrift comms From: "Michael Berman" To: "Michael Berman" Cc: "John Vines" , "Christopher Tubbs" , "accumulo" Date: Tue, 05 Nov 2013 19:44:58 -0000 Message-ID: <20131105194458.30022.48419@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Michael Berman" X-ReviewGroup: accumulo X-ReviewRequest-URL: https://reviews.apache.org/r/14972/ X-Sender: "Michael Berman" References: <20131101213720.8090.66221@reviews.apache.org> In-Reply-To: <20131101213720.8090.66221@reviews.apache.org> Reply-To: "Michael Berman" --===============4605459605209041043== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit > On Nov. 1, 2013, 9:37 p.m., Christopher Tubbs wrote: > > core/src/test/java/org/apache/accumulo/core/conf/ClientConfigurationTest.java, line 38 > > > > > > Should test expected config before and after serialization. Otherwise, if it's only correct after deserialization, we won't catch it. Well, if it's not correct before serialization, the previous unit test will fail. I don't have a problem sticking in a sanity check test of the pre-serialization config, but I guess it depends how unitary we want our unit tests to be. - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14972/#review28034 ----------------------------------------------------------- On Oct. 31, 2013, 2:35 p.m., John Vines wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14972/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2013, 2:35 p.m.) > > > Review request for accumulo and Michael Berman. > > > Bugs: ACCUMULO-1009 > https://issues.apache.org/jira/browse/ACCUMULO-1009 > > > Repository: accumulo > > > Description > ------- > > Michael Berman's October 13 patch for ACCUMULO-1009 > > > Diffs > ----- > > .gitignore 1ffa452 > core/src/main/java/org/apache/accumulo/core/cli/ClientOpts.java 9247d56 > core/src/main/java/org/apache/accumulo/core/client/ZooKeeperInstance.java 5b5d041 > core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java bb5987d > 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/AccumuloInputFormat.java bbbd0c3 > 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/mapred/InputFormatBase.java c796cd2 > core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloInputFormat.java 1cbb606 > 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/InputFormatBase.java 13f9708 > core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java 73405c5 > core/src/main/java/org/apache/accumulo/core/conf/AccumuloConfiguration.java 28cb0bd > core/src/main/java/org/apache/accumulo/core/conf/ClientConfiguration.java PRE-CREATION > core/src/main/java/org/apache/accumulo/core/conf/Property.java b6fbdd2 > 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/security/ssl/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 4140c8c > core/src/main/java/org/apache/accumulo/core/util/shell/ShellOptionsJC.java cb1f1c8 > 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 > minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloCluster.java 77776df > minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java 0b6c42c > minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloInstance.java 540d7ae > minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterTest.java 3e749ab > server/src/main/java/org/apache/accumulo/server/cli/ClientOnDefaultTable.java 53f5ac2 > server/src/main/java/org/apache/accumulo/server/cli/ClientOnRequiredTable.java e9e9bf1 > server/src/main/java/org/apache/accumulo/server/cli/ClientOpts.java 6f3516a > server/src/main/java/org/apache/accumulo/server/client/BulkImporter.java a04765f > server/src/main/java/org/apache/accumulo/server/gc/SimpleGarbageCollector.java 817aa74 > server/src/main/java/org/apache/accumulo/server/util/TServerUtils.java 1df17fe > server/src/main/resources/web/flot/jquery.flot.js aabc544 > test/pom.xml 8343bb2 > test/src/test/java/org/apache/accumulo/test/ConditionalWriterTest.java 633ea76 > test/src/test/java/org/apache/accumulo/test/ShellServerIT.java 4fbd293 > test/src/test/java/org/apache/accumulo/test/functional/AbstractMacIT.java f1a651d > test/src/test/java/org/apache/accumulo/test/functional/BulkIT.java 607f2a5 > test/src/test/java/org/apache/accumulo/test/functional/ConcurrencyIT.java c3d3160 > test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java c34fff5 > test/src/test/java/org/apache/accumulo/test/functional/MapReduceIT.java 69825bc > test/src/test/java/org/apache/accumulo/test/functional/ShutdownIT.java 8d58821 > test/src/test/java/org/apache/accumulo/test/functional/SimpleMacIT.java 4a37d82 > 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/functional/SslWithJsseIT.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/14972/diff/ > > > Testing > ------- > > > Thanks, > > John Vines > > --===============4605459605209041043==--