[ 
https://issues.apache.org/jira/browse/HADOOP-17485?focusedWorklogId=589699&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-589699
 ]

ASF GitHub Bot logged work on HADOOP-17485:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 27/Apr/21 12:21
            Start Date: 27/Apr/21 12:21
    Worklog Time Spent: 10m 
      Work Description: amahussein commented on pull request #2631:
URL: https://github.com/apache/hadoop/pull/2631#issuecomment-827562798


   > `Inconsistent synchronization of 
org.apache.hadoop.security.LdapGroupsMapping.bindPassword; locked 75% of time 
Unsynchronized access at LdapGroupsMapping.java:75% of time Unsynchronized 
access at LdapGroupsMapping.java:[line 596]`
   > Was this caused by this PR or was it already there before/
   
   I think this PR added `public Set<String> getGroupsSet(String user)` in line 
[LdapGroupsMapping#627](https://github.com/apache/hadoop/pull/2631/files#diff-34cbaed59eb28984bdb38185b850527c6bc2088d2295fee6657333084aeab668R627)
 that is not synchronized which in turn calls `doGetGroups()`. Perhaps this is 
why `findbugs` finds it a concurrency issue.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 589699)
    Time Spent: 50m  (was: 40m)

> port UGI#getGroupsSet optimizations into 2.10
> ---------------------------------------------
>
>                 Key: HADOOP-17485
>                 URL: https://issues.apache.org/jira/browse/HADOOP-17485
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 2.10.1
>            Reporter: Ahmed Hussein
>            Assignee: Ahmed Hussein
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> HADOOP-17079 introduced an optimization adding a UGI#getGroupsSet and use 
> Set#contains() instead of List#contains() to speed up large group look up 
> while minimize List->Set conversions in Groups#getGroups() call.
> This ticket is to port the changes into branch-2.10.
>  
> CC: [~Jim_Brennan], [~xyao]



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

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to