-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57219/
-----------------------------------------------------------
(Updated March 2, 2017, 1:48 a.m.)
Review request for sentry, Misha Dmitriev, Lei Xu, Hao Hao, kalyan kumar
kalvagadda, Mat Crocker, Vamsee Yarlagadda, and Vadim Spector.
Changes
-------
Addressed code review comments from Misha.
Switched from using single-element blocking queue to binary semaphore
Bugs: SENTRY-1601
https://issues.apache.org/jira/browse/SENTRY-1601
Repository: sentry
Description
-------
SENTRY-1601 Implement HMS Notification barrier on the server side
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
b9272bc80ea473685b90f6c30dc8e0b0dff7b9a9
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryMetrics.java
3f7542c2b7bfacc99b5a2bed14f841d5ec89f6e1
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java
30e91ae05af5350ea30f08e099d73feac94c120a
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/CounterWait.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java
bdbb0cc3ff639bf2e5c3725e6ebf1cc641c01374
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestCounterWait.java
PRE-CREATION
Diff: https://reviews.apache.org/r/57219/diff/2/
Changes: https://reviews.apache.org/r/57219/diff/1-2/
Testing
-------
The new CounterWait class has a unit test. End to end functionality will be
tested once HMS part is implemented.
Thanks,
Alexander Kolbasov
