-----------------------------------------------------------
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

Reply via email to