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 96C8717F4C for ; Fri, 10 Apr 2015 16:20:28 +0000 (UTC) Received: (qmail 39451 invoked by uid 500); 10 Apr 2015 16:20:28 -0000 Delivered-To: apmail-accumulo-dev-archive@accumulo.apache.org Received: (qmail 39413 invoked by uid 500); 10 Apr 2015 16:20:28 -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 39401 invoked by uid 99); 10 Apr 2015 16:20:28 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 10 Apr 2015 16:20:28 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id B04181D754B; Fri, 10 Apr 2015 16:20:29 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============0359489483669414424==" MIME-Version: 1.0 Subject: Review Request 33085: ACCUMULO-3714 Replace PropertyFilter From: "Bob Thorman" To: "accumulo" , "Bob Thorman" Date: Fri, 10 Apr 2015 16:20:29 -0000 Message-ID: <20150410162029.1489.85888@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Bob Thorman" X-ReviewGroup: accumulo X-ReviewRequest-URL: https://reviews.apache.org/r/33085/ X-Sender: "Bob Thorman" Reply-To: "Bob Thorman" X-ReviewRequest-Repository: accumulo --===============0359489483669414424== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33085/ ----------------------------------------------------------- Review request for accumulo. Repository: accumulo Description ------- * Replace internal PropertyFilter with Guava's Predicate. * Preserve MockConfiguration public API by deprecating PropertyFilter ACCUMULO-3719 Add StrictHostKeyChecking=no to default SSH options For testing purposes, strict host key verification is probably overkill. Disable it by default. Users can still override this on their own if they are concerned about security implications. Merge branch '1.6' Merge remote-tracking branch 'github/ACCUMULO-3714' ACCUMULO-2131 while statement used as if statement. Changed the while to an if. Signed-off-by: Christopher Tubbs ACCUMULO-3718 make Mutation#hashCode and Mutation#equals not change the state of the mutation Merge branch '1.5' into 1.6 Merge branch '1.6' Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/accumulo into ACCUMULO-3652 Diffs ----- core/src/main/java/org/apache/accumulo/core/cli/ClientOpts.java 6185419f0f0913b7a875cbdc608339881f4e77dd core/src/main/java/org/apache/accumulo/core/client/impl/ClientContext.java 9257a1ffbd0e216953c687949d16a6bfe2a41e3c core/src/main/java/org/apache/accumulo/core/client/mock/MockConfiguration.java 8c57c5e6c2868d10e73b41cbb1b1f94fc64eb4b5 core/src/main/java/org/apache/accumulo/core/conf/AccumuloConfiguration.java 14d68c203497ea541db61c59562b004372aed085 core/src/main/java/org/apache/accumulo/core/conf/ConfigurationCopy.java 8326725e53ce18847f9dd920a2bf8076c6469ca3 core/src/main/java/org/apache/accumulo/core/conf/DefaultConfiguration.java 17364a7662391842f33b0a29abe84e22a30ef4a9 core/src/main/java/org/apache/accumulo/core/conf/SiteConfiguration.java b410cf72e7c4f062b3607170d25ce89ce07dd35e core/src/main/java/org/apache/accumulo/core/data/Mutation.java ed51204279db2d194691b74b2885ada0cf3096e9 core/src/main/java/org/apache/accumulo/core/iterators/system/SourceSwitchingIterator.java 6c401762e4ce441753b3a04ba00481ba2a333011 core/src/test/java/org/apache/accumulo/core/client/impl/ClientContextTest.java 494eb50d4feee985269fa286bdfc869f5e278e0e core/src/test/java/org/apache/accumulo/core/conf/DefaultConfigurationTest.java e21db3d19d7a1bfad59a53d7a753a52ae27c0025 core/src/test/java/org/apache/accumulo/core/conf/ObservableConfigurationTest.java b92fac6fb7692dc70e42eca7fca46ffd37f5a48b core/src/test/java/org/apache/accumulo/core/conf/SiteConfigurationTest.java f54adb15054cd0d53938ad383fbc20f4790279ac core/src/test/java/org/apache/accumulo/core/data/MutationTest.java 6607c6e841b51b21ad5c326856e594b192a88aab minicluster/src/main/java/org/apache/accumulo/cluster/RemoteShellOptions.java 714096e16ade1ef44d1888c603c80ec11fc26a71 server/base/src/main/java/org/apache/accumulo/server/conf/NamespaceConfiguration.java bae694bdcb41874be37f0cb1d57f7e96118cd100 server/base/src/main/java/org/apache/accumulo/server/conf/TableConfiguration.java 259f2a2d7f197aa2766a1a0ac69bfa00cc469d09 server/base/src/main/java/org/apache/accumulo/server/conf/ZooCachePropertyAccessor.java bec30d299f0675d67dbb156ea05ed4d9f6817f39 server/base/src/main/java/org/apache/accumulo/server/conf/ZooConfiguration.java a53350ba285b3b2029c99c6faa38d50f37877a30 server/base/src/main/java/org/apache/accumulo/server/fs/PreferredVolumeChooser.java 77026e6af9da9958faf33daec56deced8658fd59 server/base/src/test/java/org/apache/accumulo/server/conf/NamespaceConfigurationTest.java 6bd6424242d012a163ed22b0c6f652e879c0336d server/base/src/test/java/org/apache/accumulo/server/conf/TableConfigurationTest.java 68ee2b98bdda332a2c8aa5a432ac1aee03f62d40 server/base/src/test/java/org/apache/accumulo/server/conf/ZooCachePropertyAccessorTest.java 210d24047b04d9a1692204bb9b4e4b7b984d1019 server/base/src/test/java/org/apache/accumulo/server/util/FileUtilTest.java 5fe35f9f43331f49b59bf66004b09d2148ee0888 test/src/test/java/org/apache/accumulo/test/util/CertUtils.java 3bbd7377a787585e0bf1102c43075a86243f3a71 Diff: https://reviews.apache.org/r/33085/diff/ Testing ------- Thanks, Bob Thorman --===============0359489483669414424==--