gvprathyusha6 commented on code in PR #5307:
URL: https://github.com/apache/hbase/pull/5307#discussion_r1596623746


##########
hbase-mapreduce/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java:
##########
@@ -2308,7 +2316,7 @@ protected byte[] generateRow(final int i) {
     }
 
     @Override
-    boolean testRow(final int i, final long startTime) throws IOException {
+    protected boolean testRow(final int i, final long startTime) throws 
IOException {

Review Comment:
   Weirdly I am able to override PerformanceEvaluation.TableTest.testRow, but 
not PerformanceEvaluation.SequentialWriteTest.testRow when I put all impls in 
same package name.



-- 
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: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to