[ https://issues.apache.org/jira/browse/SENTRY-2064?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16261047#comment-16261047 ]
Na Li edited comment on SENTRY-2064 at 11/21/17 4:59 PM: --------------------------------------------------------- The proposed Thrift changes {code} const string UNUSED_TIME_DATASTORE = "0001-01-01 01:01:01"; struct TPathsUpdate { 1: required bool hasFullImage; 2: optional TPathsDump pathsDump; 3: required i64 seqNum; 4: required list<TPathChanges> pathChanges; 5: optional i64 imgNum = UNUSED_PATH_UPDATE_IMG_NUM; 6: optional string currTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field } struct TPermissionsUpdate { 1: required bool hasfullImage; 2: required i64 seqNum; 3: required map<string, TPrivilegeChanges> privilegeChanges; 4: required map<string, TRoleChanges> roleChanges; 5: required string currTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field } struct TAuthzUpdateRequest { 1: required i64 permSeqNum; 2: required i64 pathSeqNum; 3: required i64 pathImgNum; 4: required string permLastTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field 5: required string pathLastTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field } {code} was (Author: linaataustin): The proposed Thrift changes const string UNUSED_TIME_DATASTORE = "0001-01-01 01:01:01"; struct TPathsUpdate { 1: required bool hasFullImage; 2: optional TPathsDump pathsDump; 3: required i64 seqNum; 4: required list<TPathChanges> pathChanges; 5: optional i64 imgNum = UNUSED_PATH_UPDATE_IMG_NUM; 6: optional string currTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field } struct TPermissionsUpdate { 1: required bool hasfullImage; 2: required i64 seqNum; 3: required map<string, TPrivilegeChanges> privilegeChanges; 4: required map<string, TRoleChanges> roleChanges; 5: required string currTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field } struct TAuthzUpdateRequest { 1: required i64 permSeqNum; 2: required i64 pathSeqNum; 3: required i64 pathImgNum; 4: required string permLastTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field 5: required string pathLastTimeAtDataStore = UNUSED_TIME_DATASTORE; <- new field } > Sentry client at HDFS should send back the timestamp in last response > --------------------------------------------------------------------- > > Key: SENTRY-2064 > URL: https://issues.apache.org/jira/browse/SENTRY-2064 > Project: Sentry > Issue Type: Sub-task > Components: Sentry > Reporter: Na Li > Assignee: Na Li > Fix For: 2.0.0 > > > When sentry server sends response to HDFS, it includes the timestamp of the > current time at the DB server. > When sentry client at HDFS gets response from sentry, it sets its local > variable, which is the latest timestamp received from sentry, to be the > timestamp in the response > When sentry client sends request, it should include the latest timestamp > received from sentry in the request. -- This message was sent by Atlassian JIRA (v6.4.14#64029)