----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46548/ -----------------------------------------------------------
Review request for sentry, Sravya Tirukkovalur and Vamsee Yarlagadda. Repository: sentry Description ------- When testing the tool from SENTRY-480 on a kerberos cluster I ran into a couple of small bugs: 1) The client is created after the UGI is returned, so may be out of date 2) The compat checker always throws an exception; this exception will list zero errors, but it would cleaner if it only threw exceptions on errors. Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SentryConfigToolSolr.java bf91f5260312644a15735a2174e88d5a0e5e61b6 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/tools/TestSentryConfigToolSolr.java 7149f17ae11b96fad279a652a2ea7d7848e97d59 Diff: https://reviews.apache.org/r/46548/diff/ Testing ------- Ran the new test. Thanks, Gregory Chanan
