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 Authored: Fri Jun 17 10:58:43 2016 -0700 Committer: Anne Yu 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.