[ 
https://issues.apache.org/jira/browse/SENTRY-1643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15954435#comment-15954435
 ] 

Alexander Kolbasov commented on SENTRY-1643:
--------------------------------------------

I was wrong, we do depend on the fact that there are no holes somewhere in the 
HDFS sync logic, so this would need to change. To keep things simple we need to 
preserve the +1 semantics.

[~hahao] can you comment on this?

> AutoIncrement ChangeID of MSentryPermChange/MSentryPathChange may be 
> error-prone
> --------------------------------------------------------------------------------
>
>                 Key: SENTRY-1643
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1643
>             Project: Sentry
>          Issue Type: Sub-task
>          Components: Hdfs Plugin
>    Affects Versions: sentry-ha-redesign
>            Reporter: Hao Hao
>            Assignee: Lei (Eddy) Xu
>            Priority: Critical
>             Fix For: sentry-ha-redesign
>
>
> In MSentryPermChange/MSentryPathChange table, the changeID field is 
> auto-increment. 
> {noformat}
> <field name="changeID" primary-key="true" value-strategy="increment">
>   <column name="CHANGE_ID" jdbc-type="BIGINT" allows-null="false"/>
> </field>{noformat}
> However, found through the integration unit test TestHDFSIntegration, the 
> value does not seem to be correctly auto increased. e.g Instead of increasing 
> by one for each new entry, it increased by some unexpected number.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to