Buildbot success in on jekyll_websites

2021-05-02 Thread buildbot
The Buildbot has detected a passing build on builder jekyll_websites while 
building accumulo.
Full details are available at:
https://ci2.apache.org/#builders/7/builds/474

Buildbot URL: https://ci2.apache.org/

Worker for this Build: bb_slave10_ubuntu

Build Reason: Triggered jekyll auto-build via .asf.yaml by edcoleman
Blamelist: asfinfra, commits@accumulo.apache.org

Build succeeded!

Sincerely,
 -The Buildbot



[accumulo-website] branch asf-staging updated: Automatic Site Publish by Buildbot

2021-05-02 Thread git-site-role
This is an automated email from the ASF dual-hosted git repository.

git-site-role pushed a commit to branch asf-staging
in repository https://gitbox.apache.org/repos/asf/accumulo-website.git


The following commit(s) were added to refs/heads/asf-staging by this push:
 new 1f547a7  Automatic Site Publish by Buildbot
1f547a7 is described below

commit 1f547a76038177862aa951e9a9e35d492de3bb55
Author: buildbot 
AuthorDate: Sun May 2 18:39:00 2021 +

Automatic Site Publish by Buildbot
---
 output/feed.xml  | 4 ++--
 output/people/index.html | 5 +
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/output/feed.xml b/output/feed.xml
index 907bf98..6f29998 100644
--- a/output/feed.xml
+++ b/output/feed.xml
@@ -6,8 +6,8 @@
 
 https://accumulo.apache.org/
 https://accumulo.apache.org/feed.xml"; rel="self" 
type="application/rss+xml"/>
-Sun, 02 May 2021 16:53:07 +
-Sun, 02 May 2021 16:53:07 +
+Sun, 02 May 2021 18:38:55 +
+Sun, 02 May 2021 18:38:55 +
 Jekyll v4.2.0
 
 
diff --git a/output/people/index.html b/output/people/index.html
index a68e8f2..f3252c7 100644
--- a/output/people/index.html
+++ b/output/people/index.html
@@ -440,6 +440,11 @@ $(function() {
   https://www.timeanddate.com/time/zones/pkt";>PKT
 
 
+  Amisha Sahu
+   
+  https://www.timeanddate.com/time/zones/ist-ireland";>IST
+
+
   Andrew George Wells
   http://clearedgeit.com";>ClearEdgeIT
   https://www.timeanddate.com/time/zones/et";>ET


[accumulo-website] branch main updated: Update contributors list to add Amisha Sahu (#280)

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo-website.git


The following commit(s) were added to refs/heads/main by this push:
 new 67386aa  Update contributors list to add Amisha Sahu (#280)
67386aa is described below

commit 67386aa0f67cb2e10f235a16b271e4b6a9c4e15a
Author: Amisha Sahu <58816552+amisha...@users.noreply.github.com>
AuthorDate: Mon May 3 00:08:32 2021 +0530

Update contributors list to add Amisha Sahu (#280)

Fixed issue #2066 with PR #2071 - made map final in MapCounter.java
---
 pages/people.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pages/people.md b/pages/people.md
index 58da208..e1fa465 100644
--- a/pages/people.md
+++ b/pages/people.md
@@ -73,6 +73,7 @@ GitHub also has a [contributor list][github-contributors] 
based on commits.
 | Al Krinker  |
   | [ET][ET]  |
 | Alex Moundalexis| [Cloudera][CLOUDERA]   
   | [ET][ET]  |
 | Ali Mustafa | [FAST-NU][FAST-NU] 
   | [PKT][PKT]|
+| Amisha Sahu |
   | [IST][IST]|
 | Andrew George Wells | [ClearEdgeIT][CLEAREDGE]   
   | [ET][ET]  |
 | Arshak Navruzyan| [Argyle Data][ARGYLE]  
   |   |
 | Ben Kelly   | [Microsoft][MICROSOFT] 
   | [GMT][GMT]/[IST][IST] |


[accumulo] branch main updated: Fix #2053 Fix typo -in TabletServerResourceManager

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
 new 7778203  Fix #2053 Fix typo -in TabletServerResourceManager
7778203 is described below

commit 77782030dbe674accf2c7d221b193e9593ae31b0
Author: Himanshu 
AuthorDate: Sun May 2 23:04:46 2021 +0530

Fix #2053 Fix typo -in TabletServerResourceManager

Rename summaryParitionPool to summaryPartitionPool
---
 .../org/apache/accumulo/tserver/TabletServerResourceManager.java  | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServerResourceManager.java
 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServerResourceManager.java
index 941a654..efd3c67 100644
--- 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServerResourceManager.java
+++ 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServerResourceManager.java
@@ -98,7 +98,7 @@ public class TabletServerResourceManager {
   private final ExecutorService assignmentPool;
   private final ExecutorService assignMetaDataPool;
   private final ExecutorService summaryRetrievalPool;
-  private final ExecutorService summaryParitionPool;
+  private final ExecutorService summaryPartitionPool;
   private final ExecutorService summaryRemotePool;
 
   private final Map scanExecutors;
@@ -363,11 +363,11 @@ public class TabletServerResourceManager {
 () -> 
context.getConfiguration().getCount(Property.TSERV_SUMMARY_REMOTE_THREADS),
 "summary remote", (ThreadPoolExecutor) summaryRemotePool);
 
-summaryParitionPool =
+summaryPartitionPool =
 ThreadPools.createExecutorService(acuConf, 
Property.TSERV_SUMMARY_PARTITION_THREADS);
 modifyThreadPoolSizesAtRuntime(
 () -> 
context.getConfiguration().getCount(Property.TSERV_SUMMARY_PARTITION_THREADS),
-"summary partition", (ThreadPoolExecutor) summaryParitionPool);
+"summary partition", (ThreadPoolExecutor) summaryPartitionPool);
 
 Collection scanExecCfg = acuConf.getScanExecutors();
 Map> scanExecQueues = new HashMap<>();
@@ -859,7 +859,7 @@ public class TabletServerResourceManager {
   }
 
   public ExecutorService getSummaryPartitionExecutor() {
-return summaryParitionPool;
+return summaryPartitionPool;
   }
 
   public ExecutorService getSummaryRemoteExecutor() {


Buildbot success in on jekyll_websites

2021-05-02 Thread buildbot
The Buildbot has detected a passing build on builder jekyll_websites while 
building accumulo.
Full details are available at:
https://ci2.apache.org/#builders/7/builds/473

Buildbot URL: https://ci2.apache.org/

Worker for this Build: bb_slave10_ubuntu

Build Reason: Triggered jekyll auto-build via .asf.yaml by edcoleman
Blamelist: asfinfra, commits@accumulo.apache.org

Build succeeded!

Sincerely,
 -The Buildbot



[accumulo-website] branch asf-staging updated: Automatic Site Publish by Buildbot

2021-05-02 Thread git-site-role
This is an automated email from the ASF dual-hosted git repository.

git-site-role pushed a commit to branch asf-staging
in repository https://gitbox.apache.org/repos/asf/accumulo-website.git


The following commit(s) were added to refs/heads/asf-staging by this push:
 new c460d55  Automatic Site Publish by Buildbot
c460d55 is described below

commit c460d55c14e3c5e458887895e430c7b57b3d425c
Author: buildbot 
AuthorDate: Sun May 2 16:53:12 2021 +

Automatic Site Publish by Buildbot
---
 output/feed.xml  | 4 ++--
 output/people/index.html | 5 +
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/output/feed.xml b/output/feed.xml
index 843ce56..907bf98 100644
--- a/output/feed.xml
+++ b/output/feed.xml
@@ -6,8 +6,8 @@
 
 https://accumulo.apache.org/
 https://accumulo.apache.org/feed.xml"; rel="self" 
type="application/rss+xml"/>
-Sat, 01 May 2021 16:26:54 +
-Sat, 01 May 2021 16:26:54 +
+Sun, 02 May 2021 16:53:07 +
+Sun, 02 May 2021 16:53:07 +
 Jekyll v4.2.0
 
 
diff --git a/output/people/index.html b/output/people/index.html
index 5a6ee44..a68e8f2 100644
--- a/output/people/index.html
+++ b/output/people/index.html
@@ -575,6 +575,11 @@ $(function() {
   https://www.timeanddate.com/time/zones/et";>ET
 
 
+  Hasan Gürcan
+   
+  https://www.timeanddate.com/time/zones/cest";>CEST
+
+
   Hayden Marchant
    
    


[accumulo-website] branch main updated: Update people.md

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo-website.git


The following commit(s) were added to refs/heads/main by this push:
 new 8fcd22c  Update people.md
 new 4319d88  Merge pull request #279 from hguercan/patch-1
8fcd22c is described below

commit 8fcd22c8da08ca46c20331631ffeca85969faeb8
Author: hguercan <72599474+hguer...@users.noreply.github.com>
AuthorDate: Sun May 2 18:25:52 2021 +0200

Update people.md
---
 pages/people.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pages/people.md b/pages/people.md
index 84e2fc3..58da208 100644
--- a/pages/people.md
+++ b/pages/people.md
@@ -100,6 +100,7 @@ GitHub also has a [contributor list][github-contributors] 
based on commits.
 | Eugene Cheipesh |
   |   |
 | Gary Singh  | [Sabre Engineering][SABRE] 
   | [ET][ET]  |
 | Harjit Singh|
   | [ET][ET]  |
+| Hasan Gürcan|
   | [CEST][CEST]  |
 | Hayden Marchant |
   |   |
 | Hung Pham   | [Cloudera][CLOUDERA]   
   | [ET][ET]  |
 | Jacob Meisler   | [Booz Allen Hamilton][BOOZ]
   | [ET][ET]  |


[accumulo] 02/02: #2040: replace immutable HashSet by Set.of() to reduce code * set locale in NumUtilTest to US * adjusted imports * closes #2040

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git

commit 4f35e6aff1763d4b19d1e93036260094b47cc589
Author: hguercan 
AuthorDate: Sat May 1 16:50:39 2021 +0200

#2040: replace immutable HashSet by Set.of() to reduce code
* set locale in NumUtilTest to US
* adjusted imports
* closes #2040
---
 .../server/security/handler/KerberosAuthenticator.java|  7 ++-
 .../org/apache/accumulo/server/util/FileSystemMonitor.java|  8 ++--
 .../accumulo/shell/commands/DeleteTableCommandTest.java   |  9 -
 test/src/main/java/org/apache/accumulo/test/OrIteratorIT.java |  3 +--
 .../src/main/java/org/apache/accumulo/test/ShellServerIT.java |  6 +++---
 .../org/apache/accumulo/test/functional/ManagerMetricsIT.java | 11 ---
 .../org/apache/accumulo/test/functional/PermissionsIT.java|  3 +--
 7 files changed, 17 insertions(+), 30 deletions(-)

diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/security/handler/KerberosAuthenticator.java
 
b/server/base/src/main/java/org/apache/accumulo/server/security/handler/KerberosAuthenticator.java
index ed66fc7..cebdbaa 100644
--- 
a/server/base/src/main/java/org/apache/accumulo/server/security/handler/KerberosAuthenticator.java
+++ 
b/server/base/src/main/java/org/apache/accumulo/server/security/handler/KerberosAuthenticator.java
@@ -20,7 +20,6 @@ package org.apache.accumulo.server.security.handler;
 
 import static java.nio.charset.StandardCharsets.UTF_8;
 
-import java.util.Arrays;
 import java.util.Base64;
 import java.util.HashSet;
 import java.util.Set;
@@ -44,15 +43,13 @@ import org.apache.zookeeper.KeeperException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import com.google.common.collect.Sets;
-
 public class KerberosAuthenticator implements Authenticator {
   private static final Logger log = 
LoggerFactory.getLogger(KerberosAuthenticator.class);
 
   private static final Set> 
SUPPORTED_TOKENS =
-  Sets.newHashSet(Arrays.asList(KerberosToken.class, SystemToken.class));
+  Set.of(KerberosToken.class, SystemToken.class);
   private static final Set SUPPORTED_TOKEN_NAMES =
-  Sets.newHashSet(KerberosToken.class.getName(), 
SystemToken.class.getName());
+  Set.of(KerberosToken.class.getName(), SystemToken.class.getName());
 
   private final ZKAuthenticator zkAuthenticator = new ZKAuthenticator();
   private ZooCache zooCache;
diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/util/FileSystemMonitor.java
 
b/server/base/src/main/java/org/apache/accumulo/server/util/FileSystemMonitor.java
index 805b3c8..39ce84a 100644
--- 
a/server/base/src/main/java/org/apache/accumulo/server/util/FileSystemMonitor.java
+++ 
b/server/base/src/main/java/org/apache/accumulo/server/util/FileSystemMonitor.java
@@ -25,9 +25,7 @@ import java.io.File;
 import java.io.FileReader;
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.HashMap;
-import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
@@ -41,15 +39,13 @@ import org.apache.accumulo.core.util.threads.Threads;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import com.google.common.collect.Sets;
-
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 public class FileSystemMonitor {
   private static final String PROC_MOUNTS = "/proc/mounts";
   private static final Logger log = 
LoggerFactory.getLogger(FileSystemMonitor.class);
   private static final String DEVICE_PREFIX = "/dev/";
-  private static final Set ACCEPTED_FILESYSTEMS = 
Sets.newHashSet("ext3", "ext4", "xfs");
+  private static final Set ACCEPTED_FILESYSTEMS = Set.of("ext3", 
"ext4", "xfs");
 
   static class Mount {
 String device;
@@ -64,7 +60,7 @@ public class FileSystemMonitor {
   mountPoint = tokens[1].trim();
   filesystemType = tokens[2].trim().toLowerCase();
 
-  options = new HashSet<>(Arrays.asList(tokens[3].split(",")));
+  options = Set.of(tokens[3].split(","));
 }
 
 @Override
diff --git 
a/shell/src/test/java/org/apache/accumulo/shell/commands/DeleteTableCommandTest.java
 
b/shell/src/test/java/org/apache/accumulo/shell/commands/DeleteTableCommandTest.java
index 34a8c2b..a977091 100644
--- 
a/shell/src/test/java/org/apache/accumulo/shell/commands/DeleteTableCommandTest.java
+++ 
b/shell/src/test/java/org/apache/accumulo/shell/commands/DeleteTableCommandTest.java
@@ -20,24 +20,23 @@ package org.apache.accumulo.shell.commands;
 
 import static org.junit.Assert.assertEquals;
 
-import java.util.Arrays;
-import java.util.HashSet;
 import java.util.Set;
 
 import org.apache.accumulo.core.metadata.MetadataTable;
 import org.apache.accumulo.core.metadata.RootTable;
 import org.junit.Test;
 
+import com.google.common.collect.Sets;
+
 public class DeleteTableCommandTest {
 
   @Test
   public void 

[accumulo] branch main updated (f35ca73 -> 4f35e6a)

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git.


from f35ca73  Fix NumUtilTests to set Locale to US (#2072)
 new c74e2ea  #2040: replace immutable HashSet by Set.of() to reduce code * 
adjusted imports * closes #2040
 new 4f35e6a  #2040: replace immutable HashSet by Set.of() to reduce code * 
set locale in NumUtilTest to US * adjusted imports * closes #2040

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../apache/accumulo/core/file/FileOperations.java  | 12 +--
 .../client/security/tokens/KerberosTokenTest.java  |  4 +---
 .../core/iterators/user/RowFilterTest.java | 25 +++---
 .../org/apache/accumulo/fate/AgeOffStoreTest.java  | 23 +---
 .../server/constraints/MetadataConstraints.java| 14 ++--
 .../security/handler/KerberosAuthenticator.java|  7 ++
 .../accumulo/server/util/FileSystemMonitor.java|  8 ++-
 .../shell/commands/DeleteTableCommandTest.java |  9 
 .../org/apache/accumulo/test/OrIteratorIT.java |  3 +--
 .../org/apache/accumulo/test/ShellServerIT.java|  6 +++---
 .../accumulo/test/functional/ManagerMetricsIT.java | 11 --
 .../accumulo/test/functional/PermissionsIT.java|  3 +--
 12 files changed, 52 insertions(+), 73 deletions(-)


[accumulo] 01/02: #2040: replace immutable HashSet by Set.of() to reduce code * adjusted imports * closes #2040

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git

commit c74e2ead8a586c5cad1f43e65eb5e318e0f6512b
Author: hguercan 
AuthorDate: Sat May 1 02:06:43 2021 +0200

#2040: replace immutable HashSet by Set.of() to reduce code
* adjusted imports
* closes #2040
---
 .../apache/accumulo/core/file/FileOperations.java  | 12 +--
 .../client/security/tokens/KerberosTokenTest.java  |  4 +---
 .../core/iterators/user/RowFilterTest.java | 25 +++---
 .../org/apache/accumulo/fate/AgeOffStoreTest.java  | 23 +---
 .../server/constraints/MetadataConstraints.java| 14 ++--
 5 files changed, 35 insertions(+), 43 deletions(-)

diff --git 
a/core/src/main/java/org/apache/accumulo/core/file/FileOperations.java 
b/core/src/main/java/org/apache/accumulo/core/file/FileOperations.java
index 53ba829..494d433 100644
--- a/core/src/main/java/org/apache/accumulo/core/file/FileOperations.java
+++ b/core/src/main/java/org/apache/accumulo/core/file/FileOperations.java
@@ -21,8 +21,6 @@ package org.apache.accumulo.core.file;
 import static 
org.apache.accumulo.core.file.blockfile.impl.CacheProvider.NULL_PROVIDER;
 
 import java.io.IOException;
-import java.util.Arrays;
-import java.util.HashSet;
 import java.util.Objects;
 import java.util.Set;
 
@@ -47,13 +45,13 @@ public abstract class FileOperations {
   private static final String HADOOP_JOBHISTORY_LOCATION = "_logs"; // dir 
related to
 // 
hadoop.job.history.user.location
 
-  private static final HashSet validExtensions =
-  new HashSet<>(Arrays.asList(Constants.MAPFILE_EXTENSION, 
RFile.EXTENSION));
+  private static final Set validExtensions =
+  Set.of(Constants.MAPFILE_EXTENSION, RFile.EXTENSION);
 
   // Sometimes we want to know what files accumulo bulk processing creates
-  private static final HashSet bulkWorkingFiles =
-  new HashSet<>(Arrays.asList(Constants.BULK_LOAD_MAPPING, 
Constants.BULK_RENAME_FILE,
-  FileOutputCommitter.SUCCEEDED_FILE_NAME, 
HADOOP_JOBHISTORY_LOCATION));
+  private static final Set bulkWorkingFiles =
+  Set.of(Constants.BULK_LOAD_MAPPING, Constants.BULK_RENAME_FILE,
+  FileOutputCommitter.SUCCEEDED_FILE_NAME, HADOOP_JOBHISTORY_LOCATION);
 
   public static Set getValidExtensions() {
 return validExtensions;
diff --git 
a/core/src/test/java/org/apache/accumulo/core/client/security/tokens/KerberosTokenTest.java
 
b/core/src/test/java/org/apache/accumulo/core/client/security/tokens/KerberosTokenTest.java
index 38bb04a..e522183 100644
--- 
a/core/src/test/java/org/apache/accumulo/core/client/security/tokens/KerberosTokenTest.java
+++ 
b/core/src/test/java/org/apache/accumulo/core/client/security/tokens/KerberosTokenTest.java
@@ -20,8 +20,6 @@ package org.apache.accumulo.core.client.security.tokens;
 
 import static org.junit.Assert.fail;
 
-import java.util.Arrays;
-import java.util.HashSet;
 import java.util.Set;
 
 import org.apache.hadoop.security.UserGroupInformation.AuthenticationMethod;
@@ -36,7 +34,7 @@ public class KerberosTokenTest {
   public void testAuthMethodAcceptance() {
 // There is also KERBEROS_SSL but that appears to be deprecated/OBE
 Set allowedMethods =
-new HashSet<>(Arrays.asList(AuthenticationMethod.KERBEROS, 
AuthenticationMethod.PROXY));
+Set.of(AuthenticationMethod.KERBEROS, AuthenticationMethod.PROXY);
 for (AuthenticationMethod authMethod : AuthenticationMethod.values()) {
   final boolean allowable = allowedMethods.contains(authMethod);
   try {
diff --git 
a/core/src/test/java/org/apache/accumulo/core/iterators/user/RowFilterTest.java 
b/core/src/test/java/org/apache/accumulo/core/iterators/user/RowFilterTest.java
index f16b814..2e45fe1 100644
--- 
a/core/src/test/java/org/apache/accumulo/core/iterators/user/RowFilterTest.java
+++ 
b/core/src/test/java/org/apache/accumulo/core/iterators/user/RowFilterTest.java
@@ -22,7 +22,6 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
 
 import java.io.IOException;
-import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashSet;
 import java.util.LinkedList;
@@ -66,14 +65,14 @@ public class RowFilterTest {
 
   // ensure that seeks are confined to the row
   rowIterator.seek(new Range(null, false, firstKey == null ? null : 
firstKey.getRow(), false),
-  new HashSet<>(), false);
+  Set.of(), false);
   while (rowIterator.hasTop()) {
 sum2 += Integer.parseInt(rowIterator.getTopValue().toString());
 rowIterator.next();
   }
 
   rowIterator.seek(new Range(firstKey == null ? null : firstKey.getRow(), 
false, null, true),
-  new HashSet<>(), false);
+  Set.of(), false);
   while (rowIterator.hasTop()) {
 sum2 += Integer.pars

[accumulo] branch main updated: Fix NumUtilTests to set Locale to US (#2072)

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
 new f35ca73  Fix NumUtilTests to set Locale to US (#2072)
f35ca73 is described below

commit f35ca738a018004fa117fca216ede2f46df16dcf
Author: EdColeman 
AuthorDate: Sun May 2 10:27:23 2021 -0400

Fix NumUtilTests to set Locale to US (#2072)

- Identified in PR#2040 by hguercan, the test can fail for systems that
  with a Locale different that US.
---
 core/src/test/java/org/apache/accumulo/core/util/NumUtilTest.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/core/src/test/java/org/apache/accumulo/core/util/NumUtilTest.java 
b/core/src/test/java/org/apache/accumulo/core/util/NumUtilTest.java
index 6bf5c1f..174ddb8 100644
--- a/core/src/test/java/org/apache/accumulo/core/util/NumUtilTest.java
+++ b/core/src/test/java/org/apache/accumulo/core/util/NumUtilTest.java
@@ -22,12 +22,15 @@ import static 
org.apache.accumulo.core.util.NumUtil.bigNumberForQuantity;
 import static org.apache.accumulo.core.util.NumUtil.bigNumberForSize;
 import static org.junit.Assert.assertEquals;
 
+import java.util.Locale;
+
 import org.junit.Test;
 
 public class NumUtilTest {
 
   @Test
   public void testBigNumberForSize() {
+Locale.setDefault(Locale.US);
 assertEquals("1,000", bigNumberForSize(1000));
 assertEquals("1.00K", bigNumberForSize(1024));
 assertEquals("1.50K", bigNumberForSize(1024 + (1024 / 2)));


[accumulo] branch main updated: Fixes #2066 - Declaration of map in MapCounter.java as final (#2071)

2021-05-02 Thread edcoleman
This is an automated email from the ASF dual-hosted git repository.

edcoleman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
 new 17c0174  Fixes #2066 - Declaration of map in MapCounter.java as final 
(#2071)
17c0174 is described below

commit 17c017471a263edeb456b9a60d8d10b89f60ce85
Author: Amisha Sahu <58816552+amisha...@users.noreply.github.com>
AuthorDate: Sun May 2 18:40:05 2021 +0530

Fixes #2066 - Declaration of map in MapCounter.java as final (#2071)

Fixes #2066
---
 core/src/main/java/org/apache/accumulo/core/util/MapCounter.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/java/org/apache/accumulo/core/util/MapCounter.java 
b/core/src/main/java/org/apache/accumulo/core/util/MapCounter.java
index 80aac5f..3b77aa3 100644
--- a/core/src/main/java/org/apache/accumulo/core/util/MapCounter.java
+++ b/core/src/main/java/org/apache/accumulo/core/util/MapCounter.java
@@ -31,7 +31,7 @@ public class MapCounter {
 long l = 0L;
   }
 
-  private HashMap map;
+  private final HashMap map;
 
   public MapCounter() {
 map = new HashMap<>();