-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57219/
-----------------------------------------------------------
(Updated March 4, 2017, 1:34 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 Vadim and Lei
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
c1186ba405a05be70336e9169a1454208df016ca
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
8b07f5b71f461213e1ef5fe7c41d7b7080d3c925
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/6/
Changes: https://reviews.apache.org/r/57219/diff/5-6/
Testing
-------
The new CounterWait class has a unit test. End to end functionality will be
tested once HMS part is implemented.
Thanks,
Alexander Kolbasov