[GitHub] [hbase] HorizonNet commented on a change in pull request #1742: HBASE-24401 Cell size limit check on append should consider 0 or less…

[GitBox <gi...@apache.org>]

HorizonNet commented on a change in pull request #1742:
URL: https://github.com/apache/hbase/pull/1742#discussion_r428208669



##########
File path: hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide5.java
##########
@@ -2274,6 +2274,21 @@ public void testCellSizeLimit() throws IOException {
         // expected
       }
     }
+    //test cell size no limit
+    final TableName noLimitTable = TableName.valueOf("testCellSizeNoLimit");

Review comment:
       Use the `@Rule` provided in this class instead of a String for the table name, such as `name.getMethodName()`.




----------------------------------------------------------------
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:
users@infra.apache.org