HorizonNet commented on a change in pull request #1830:
URL: https://github.com/apache/hbase/pull/1830#discussion_r435411310
##########
File path:
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/wal/AbstractTestWALReplay.java
##########
@@ -766,15 +766,14 @@ private int getScannedCount(RegionScanner scanner) throws
IOException {
/**
* Create an HRegion with the result of a WAL split and test we only see the
- * good edits
- * @throws Exception
+ * good edits=
Review comment:
I think it should be "." instead of a "=".
##########
File path:
hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestCatalogJanitor.java
##########
@@ -350,31 +353,20 @@ public void testSplitParentFirstComparator() {
SplitParentFirstComparator comp = new SplitParentFirstComparator();
TableDescriptor td = createTableDescriptorForCurrentMethod();
- /* Region splits:
Review comment:
I think it's better to keep the visual representation.
----------------------------------------------------------------
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]
