-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19704/
-----------------------------------------------------------

(Updated March 28, 2014, 7:12 p.m.)


Review request for sentry, Brock Noland and Prasad Mujumdar.


Changes
-------

Feedback from revision # 1.


Bugs: SENTRY-138
    https://issues.apache.org/jira/browse/SENTRY-138


Repository: sentry


Description
-------

This patch does the following,

1. Sets timestamp on service side in SentryStore.java right before persisting 
the relevant object.
2. Fixes up the table names, index names in package.jdo, which maps java 
classes to underlying tables.


Diffs (updated)
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo
 03f7549 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 9c678d5 

Diff: https://reviews.apache.org/r/19704/diff/


Testing
-------

All tests pass.


Thanks,

Shreepadma Venugopalan

Reply via email to