You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by el...@apache.org on 2014/09/16 00:52:26 UTC

[1/3] git commit: ACCUMULO-3128 Screwed up the conditional

Repository: accumulo
Updated Branches:
  refs/heads/1.6.1-SNAPSHOT 4556ec4bd -> e455005f6
  refs/heads/master 0c45fae3a -> 9eb93a15b


ACCUMULO-3128 Screwed up the conditional


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/e455005f
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/e455005f
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/e455005f

Branch: refs/heads/1.6.1-SNAPSHOT
Commit: e455005f6e9a33f82eb25d6b89816704ad8663de
Parents: 4556ec4
Author: Josh Elser <el...@apache.org>
Authored: Mon Sep 15 18:49:16 2014 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Mon Sep 15 18:49:16 2014 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/core/util/SslConnectionParams.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/e455005f/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java b/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
index 640f66e..d3b1ae9 100644
--- a/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
+++ b/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
@@ -79,8 +79,9 @@ public class SslConnectionParams  {
 
   private static String passwordFromConf(AccumuloConfiguration conf, String defaultPassword, Property passwordOverrideProperty) {
     String keystorePassword = conf.get(passwordOverrideProperty);
-    if (log.isTraceEnabled() && !keystorePassword.isEmpty()) {
-      log.trace("Using explicit SSL private key password from " + passwordOverrideProperty.getKey());
+    if (!keystorePassword.isEmpty()) {
+      if (log.isTraceEnabled())
+        log.trace("Using explicit SSL private key password from " + passwordOverrideProperty.getKey());
     } else {
       keystorePassword = defaultPassword;
     }


[2/3] git commit: ACCUMULO-3128 Screwed up the conditional

Posted by el...@apache.org.
ACCUMULO-3128 Screwed up the conditional


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/e455005f
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/e455005f
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/e455005f

Branch: refs/heads/master
Commit: e455005f6e9a33f82eb25d6b89816704ad8663de
Parents: 4556ec4
Author: Josh Elser <el...@apache.org>
Authored: Mon Sep 15 18:49:16 2014 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Mon Sep 15 18:49:16 2014 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/core/util/SslConnectionParams.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/e455005f/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java b/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
index 640f66e..d3b1ae9 100644
--- a/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
+++ b/core/src/main/java/org/apache/accumulo/core/util/SslConnectionParams.java
@@ -79,8 +79,9 @@ public class SslConnectionParams  {
 
   private static String passwordFromConf(AccumuloConfiguration conf, String defaultPassword, Property passwordOverrideProperty) {
     String keystorePassword = conf.get(passwordOverrideProperty);
-    if (log.isTraceEnabled() && !keystorePassword.isEmpty()) {
-      log.trace("Using explicit SSL private key password from " + passwordOverrideProperty.getKey());
+    if (!keystorePassword.isEmpty()) {
+      if (log.isTraceEnabled())
+        log.trace("Using explicit SSL private key password from " + passwordOverrideProperty.getKey());
     } else {
       keystorePassword = defaultPassword;
     }


[3/3] git commit: Merge branch '1.6.1-SNAPSHOT'

Posted by el...@apache.org.
Merge branch '1.6.1-SNAPSHOT'


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/9eb93a15
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/9eb93a15
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/9eb93a15

Branch: refs/heads/master
Commit: 9eb93a15bb868d637c8b26f1709945188c8e385e
Parents: 0c45fae e455005
Author: Josh Elser <el...@apache.org>
Authored: Mon Sep 15 18:49:36 2014 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Mon Sep 15 18:49:36 2014 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/core/util/SslConnectionParams.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------