dlmarion commented on code in PR #2665:
URL: https://github.com/apache/accumulo/pull/2665#discussion_r940242670
##########
core/src/test/java/org/apache/accumulo/core/conf/cluster/ClusterConfigParserTest.java:
##########
@@ -121,12 +131,21 @@ public void testShellOutput() throws Exception {
ClusterConfigParser.outputShellVariables(contents, ps);
ps.close();
- Map<String,
- String> expected = Map.of("MANAGER_HOSTS", "\"localhost1
localhost2\"", "MONITOR_HOSTS",
- "\"localhost1 localhost2\"", "GC_HOSTS", "\"localhost\"",
"TSERVER_HOSTS",
- "\"localhost1 localhost2 localhost3 localhost4\"",
"COORDINATOR_HOSTS",
- "\"localhost1 localhost2\"", "COMPACTION_QUEUES", "\"q1 q2\"",
"COMPACTOR_HOSTS_q1",
- "\"localhost1 localhost2\"", "COMPACTOR_HOSTS_q2", "\"localhost3
localhost4\"");
+ Map<String,String> expected = new HashMap<>();
+ expected.put("MANAGER_HOSTS", "localhost1 localhost2");
+ expected.put("MONITOR_HOSTS", "localhost1 localhost2");
+ expected.put("GC_HOSTS", "localhost");
+ expected.put("TSERVER_HOSTS", "localhost1 localhost2 localhost3
localhost4");
+ expected.put("COORDINATOR_HOSTS", "localhost1 localhost2");
+ expected.put("COMPACTION_QUEUES", "q1 q2");
+ expected.put("COMPACTOR_HOSTS_q1", "localhost1 localhost2");
+ expected.put("COMPACTOR_HOSTS_q2", "localhost3 localhost4");
+ expected.put("SSERVER_GROUPS", "default highmem cheap");
+ expected.put("SSERVER_HOSTS_default", "localhost1 localhost2");
+ expected.put("SSERVER_HOSTS_highmem", "hmvm1 hmvm2 hmvm3");
+ expected.put("SSERVER_HOSTS_cheap", "burstyvm1 burstyvm2");
+
+ expected = Maps.transformValues(expected, v -> '"' + v + '"');
Review Comment:
Removed use of Guava in d416900f14ac0c0e88af37aff49cb3b97a6a8173
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
