[jira] [Commented] (KYLIN-4697) User info update logic is not correct

["Zhong Yanghong (Jira)" <ji...@apache.org>]

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

Zhong Yanghong commented on KYLIN-4697:
---------------------------------------

There's one more thing we need to take care of. In HBase, the rowkey is case-sensitive. While in Kylin, the user name is case-insensitive. We need to avoid the case that there's multiple HBase records existing for the same user. Otherwise, WriteConflictException may occur when updating user info.

> User info update logic is not correct
> -------------------------------------
>
>                 Key: KYLIN-4697
>                 URL: https://issues.apache.org/jira/browse/KYLIN-4697
>             Project: Kylin
>          Issue Type: Bug
>            Reporter: Zhong Yanghong
>            Assignee: Zhong Yanghong
>            Priority: Major
>
> There are mainly two issues:
> * The logic for KylinAuthenticationProvider.needUpdateUser() is not correct due to not considering ALL_USERS
> * The logic of updateUser in some places is not correct due to not following copy on write. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)