Impala Public Jenkins has submitted this change and it was merged. ( 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. The log messages were muted in IMPALA-6878, however since Impala no longer uses the shaded version of Sentry client in IMPALA-7423, the patch in IMPALA-6878 no longer worked. This patch fixes the muting of Sentry error messages by turning off the log level using the non-shaded version of Sentry Thrift client. Testing: - Manually tested by starting Sentry and did not see any error messages logged into stdout. Change-Id: I75782d23c1cb67564a9265bf3cc94fd590c7b666 Reviewed-on: http://gerrit.cloudera.org:8080/11281 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/test/java/org/apache/impala/testutil/SentryServicePinger.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I75782d23c1cb67564a9265bf3cc94fd590c7b666 Gerrit-Change-Number: 11281 Gerrit-PatchSet: 4 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]>
