-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59720/
-----------------------------------------------------------
(Updated June 1, 2017, 10:17 p.m.)
Review request for sentry and Alexander Kolbasov.
Changes
-------
Added more changes to the SentryStore API that persists/retrieve snapshots,
add/delete/rename/update paths.
Added more unit tests to verify that all actions mentioned above work after
replacing a hive snapshot.
Bugs: SENTRY-1781
https://issues.apache.org/jira/browse/SENTRY-1781
Repository: sentry
Description
-------
This patch only adds an extra column to the MAuthzPathsMapping table that
contains an image identifier that will be used to detect full snapshots. This
patch does not increment the identifier yet, it only uses 0 for now.
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MAuthzPathsMapping.java
f51894bb7e109c37997e7134e07a82f46c0a3c44
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo
96ab462d22afaf0f3004926f2855a8bb8cabc53b
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
cb05a84cb88151156893f0f9d1885bd83cbc2abb
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
c5dddfb07bdfcb4c2bfee51f5cd7c684eff65a80
Diff: https://reviews.apache.org/r/59720/diff/2/
Changes: https://reviews.apache.org/r/59720/diff/1-2/
Testing
-------
All unit tests run locally, but there are some flaky tests on Jenkins no
related to this patch.
Thanks,
Sergio Pena
