---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52138/#review152452
---
Ship it!
Just one small nit with logging.
- Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52138/#review152450
---
There is some strangeness in the way SentryStore/DelegateSentryStore is
implemented.
DelegateSentryStore implements SentryStoreLayer interface, but SentryStore
doesn't. It looks like SentryStore should implement this as well.
Any thoughts on this?
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52138/#review152377
---
Can we add a comment that, this is a workaround until HIVE-14906
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52795/
---
Review request for sentry, Alexander Kolbasov, Anne Yu, Li Li, and Sravya
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52138/#review152359
---
Ship it!
Ship It!
- Li Li
On Oct. 12, 2016, 7:15 a.m., Hao
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52675/#review152274
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52138/
---
(Updated Oct. 12, 2016, 7:15 a.m.)
Review request for sentry, Alexander
