Fredy Wijaya has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11281


Change subject: IMPALA-7470: SentryServicePinger logs error messages on success
......................................................................

IMPALA-7470: SentryServicePinger logs error messages on success

SentryServicePinger checks if Sentry is running by calling a Sentry API
to get a list of roles. If Sentry is not yet running, an exception will
be thrown. However, the Sentry client implementation will log some error
messages when an exception is thrown. For the purpose of SentryServicePinger,
this can be too noisy and verbose and may also confuse other developers
into thinking it was a failure when starting Sentry. This patch mutes
the logging of Sentry error messages when calling a Sentry API to get a
list of roles.

Testing:
- Manually tested by starting Sentry and did not see any error messages
  logged into stdout.

Change-Id: I75782d23c1cb67564a9265bf3cc94fd590c7b666
---
M fe/src/test/java/org/apache/impala/testutil/SentryServicePinger.java
1 file changed, 3 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/81/11281/1
--
To view, visit http://gerrit.cloudera.org:8080/11281
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I75782d23c1cb67564a9265bf3cc94fd590c7b666
Gerrit-Change-Number: 11281
Gerrit-PatchSet: 1
Gerrit-Owner: Fredy Wijaya <[email protected]>

Reply via email to