----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52526/ -----------------------------------------------------------
(Updated Oct. 14, 2016, 5:06 p.m.) Review request for sentry, Alexander Kolbasov, Anne Yu, Hao Hao, and Sravya Tirukkovalur. Repository: sentry Description (updated) ------- Since currently only pool model is used, here update retry logic for pool model. For each full retry we will cycle through all available sentry servers randomly. After each full retry, we will have a small random sleep. Diffs (updated) ----- 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 f98ebd135a383ff090b1b204cc62644403310df7 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) ------- Tested in my dev with the following cases: 1. one server is down before client connection 2. one server is down after client connection and during client request Thanks, Li Li
