DomGarguilo opened a new pull request, #5488:
URL: https://github.com/apache/accumulo/pull/5488

   This fixes a bug where new resource groups were never actually added to the 
resourceMapping map. The previous code used getOrDefault() to retrieve a set, 
but if the group wasn't already present in the map, it returned a temporary set 
that was modified but never stored. As a result, those resource groups and 
their associated tables were effectively ignored. Switching to 
computeIfAbsent() ensures the set is both created and inserted into the map 
before adding the table ID.
   
   I also fixed a warning about non final String in the log message but thats 
trivial and unrelated.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org

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

Reply via email to