[
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)
