----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56892/ -----------------------------------------------------------
Review request for sentry, Alexander Kolbasov, Hao Hao, and kalyan kumar kalvagadda. Repository: sentry Description ------- Looking at the code of HMSFollower, we seem to be establishing HMS connection for all servers of Sentry servers irrespective of whether they are leader or not. Whereas the subsequent operations of HMSFollower are only limited to the leader of the Sentry group. To avoid extra connections, we can limit only the leader to get a connection to HMS. Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java f88f6f17e746f6dd4db8b8f4679a2dc9aa1e4ee0 Diff: https://reviews.apache.org/r/56892/diff/ Testing ------- Thanks, Vamsee Yarlagadda
