hguercan commented on pull request #2065:
URL: https://github.com/apache/accumulo/pull/2065#issuecomment-830646088


   Also added Locale setting in NumUtilTest to US since unit testing fails if 
LC settings for numbers are set to DE for example
   
    accumulo git:(replace_hashset_by_setof) ✗ find . -name "*.java" | xargs 
grep asList | grep new | grep Set
   
   
./core/src/main/java/org/apache/accumulo/core/clientImpl/TableOperationsImpl.java:
 Sets.copyOf but wasnt sure about null values
   ./test/src/main/java/org/apache/accumulo/test/UserCompactionStrategyIT.java: 
SortedSet --> TreeSet
   ./test/src/main/java/org/apache/accumulo/test/OrIteratorIT.java: TreeSet
   ./test/src/main/java/org/apache/accumulo/test/OrIteratorIT.java: TreeSet
   ./test/src/main/java/org/apache/accumulo/test/TableOperationsIT.java:  
TreeSet
   
./test/src/main/java/org/apache/accumulo/test/functional/ConfigurableCompactionIT.java:
 TreeSet
   ./test/src/main/java/org/apache/accumulo/test/functional/SummaryIT.java: 
TreeSet
   ./shell/src/main/java/org/apache/accumulo/shell/commands/DUCommand.java: 
SortedSet --> TreeSet
   
./server/tserver/src/main/java/org/apache/accumulo/tserver/AssignmentHandler.java:
  TreeSet
   
./server/manager/src/test/java/org/apache/accumulo/manager/replication/UnorderedWorkAssignerTest.java:
 gets modified
   
./server/base/src/main/java/org/apache/accumulo/server/rpc/TServerUtils.java: 
gets modified


-- 
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:
[email protected]


Reply via email to