EdColeman commented on code in PR #3177:
URL: https://github.com/apache/accumulo/pull/3177#discussion_r1092322151
##########
test/src/main/java/org/apache/accumulo/test/TableOperationsIT.java:
##########
@@ -205,6 +205,17 @@ public void createTableWithTableNameLengthLimit()
assertFalse(tableOps.exists(t2));
}
+ @Test
+ public void createTableWithBadProperties()
+ throws AccumuloException, AccumuloSecurityException,
TableExistsException {
+ TableOperations tableOps = accumuloClient.tableOperations();
+ String t0 = StringUtils.repeat('a', MAX_TABLE_NAME_LEN - 1);
Review Comment:
It would be nice for consistency - and the getUniqueNames uses the test and
the function in the name to allow multiple tests in the same file to avoid
collisions. Not sure if it would apply here, but that would also pop up in the
logs if anyone was trying to chase something down.
--
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]
