----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52526/ -----------------------------------------------------------
(Updated Oct. 7, 2016, 12:30 a.m.) Review request for sentry, Alexander Kolbasov, Anne Yu, Hao Hao, and Sravya Tirukkovalur. Repository: sentry Description ------- Add retry logic for non-pool model. Diffs (updated) ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClientDefaultImpl.java 4f42a51b1449fe15f856ba252103e66383e175d7 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/ThriftUtil.java 3a96d0b124c00efc99cef256c72c25f5c6168007 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/PoolClientInvocationHandler.java 842d5cafb06910fcbe6c53002f2101ec5b890a9e sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java abc3f58d21bb774427a34399b6e9f51a37ba51db sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestPoolClientInvocationHandler.java 5b0e12bbf12510d8d424aa2b7f51076a913234c5 Diff: https://reviews.apache.org/r/52526/diff/ Testing (updated) ------- In non-pool model, for each full retry we will cycle through all available sentry servers. Before each full retry, we will shuffle the server list, and after each full retry, we will have a small random sleep. Thanks, Li Li
