sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ann...@apache.org
Subject sentry git commit: SENTRY-1336: Fix PMD violation in testSentryServiceGauges (Rahul Sharma, reviewed by Anne Yu)
Date Fri, 17 Jun 2016 17:42:28 GMT
Repository: sentry
Updated Branches:
  refs/heads/sentry-ha-redesign 81990992b -> 08b2212bb


SENTRY-1336: Fix PMD violation in testSentryServiceGauges (Rahul Sharma, reviewed by Anne
Yu)


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

Branch: refs/heads/sentry-ha-redesign
Commit: 08b2212bbce3a9e672734ceeed01832fd3fdbd6e
Parents: 8199099
Author: Anne Yu <anneyu@cloudera.com>
Authored: Fri Jun 17 10:58:43 2016 -0700
Committer: Anne Yu <anneyu@cloudera.com>
Committed: Fri Jun 17 10:58:43 2016 -0700

----------------------------------------------------------------------
 .../provider/db/service/thrift/TestSentryServiceMetrics.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sentry/blob/08b2212b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceMetrics.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceMetrics.java
b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceMetrics.java
index 441a108..3fff450 100644
--- a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceMetrics.java
+++ b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceMetrics.java
@@ -44,8 +44,8 @@ public class TestSentryServiceMetrics extends SentryServiceIntegrationBase
{
     //More Cases to be added once Sentry HA is implemented
 
     //Check for gauges with the server handle.
-    Assert.assertEquals(new Boolean(false),server.getIsHAGauge().getValue());
-    Assert.assertEquals(new Boolean(true),server.getIsActiveGauge().getValue());
+    Assert.assertEquals(Boolean.FALSE,server.getIsHAGauge().getValue());
+    Assert.assertEquals(Boolean.TRUE,server.getIsActiveGauge().getValue());
   }
 
   //Overriding this method as the client handle does not exist.


Mime
View raw message