----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61983/ -----------------------------------------------------------
(Updated Aug. 30, 2017, 7:33 p.m.) Review request for sentry, Alexander Kolbasov, Arjun Mishra, kalyan kumar kalvagadda, and Na Li. Changes ------- Addressed Sasha's comments. Repository: sentry Description ------- Sentry client logic supports retry of connections to server. But it should be complemented with sufficient retry interval otherwise they fail fast without trying for a decent time. Diffs (updated) ----- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/RetryClientInvocationHandler.java d400f8999d74cbc66c19a739a05d10a201635c5b sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConfigInterface.java 9fd401316f5184a80d737c463e03291bdebb2287 sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConstants.java 358d2824b7ce55a89945d321bf1a27fdbdd71e62 sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryHDFSClientTransportConfig.java 1724e7f9610e1e65cc24bd690d73530f6fb14049 sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryPolicyClientTransportConfig.java 45522df0d5803f525c29bfa9b06fde9e3c7e5939 Diff: https://reviews.apache.org/r/61983/diff/2/ Changes: https://reviews.apache.org/r/61983/diff/1-2/ Testing ------- Sentry unit test run in progress. Thanks, Vamsee Yarlagadda
