Some minor edits on comments in the code

Author: Kadir Ozdemir

phoenix-core-client/src/main/java/org/apache/phoenix/compile/QueryCompiler.java

@@ -758,9 +758,9 @@ protected QueryPlan compileSingleFlatQuery(StatementContext context, SelectState
         throw new SQLException(
           "ROW_SIZE() can only be an argument to an aggregation function in a select clause. \n"
             + "To get the size of a single row, an aggregation function can be used over the row, e.g., \n"
-            + "SELECT SUM(ROW_SIZE) ... WHERE PK = <my PK>. To return the row sizes for multiple rows, \n"
+            + "SELECT SUM(ROW_SIZE()) ... WHERE PK = <my PK>. To return the row sizes for multiple rows, \n"
             + "a group by clause can be used to have single row groups, e.g., \n"
-            + "SELECT SUM(ROW_SIZE) ... WHERE PK = <my PK> GROUP BY PK ");
+            + "SELECT SUM(ROW_SIZE()) ... WHERE PK = <my PK> GROUP BY PK ");
       }
     }
     boolean isApplicable = true;

phoenix-core-client/src/main/java/org/apache/phoenix/filter/RowLevelFilter.java

@@ -29,7 +29,7 @@
 import org.apache.phoenix.schema.tuple.Tuple;
 
 /**
- * Filter for use when expressions reference to the entire row
+ * Filter used when expressions reference to the entire row
  */
 public class RowLevelFilter extends BooleanExpressionFilter {
   private boolean keepRow = false;
@@ -47,10 +47,6 @@ public void reset() {
     keepRow = false;
   }
 
-  /**
-   * Evaluate in filterKeyValue instead of filterRowKey, because HBASE-6562 causes filterRowKey to
-   * be called with deleted or partial row keys.
-   */
   // No @Override for HBase 3 compatibility
   public ReturnCode filterKeyValue(Cell v) {
     return filterCell(v);

phoenix-core/src/it/java/org/apache/phoenix/end2end/BaseAggregateIT.java

@@ -17,7 +17,6 @@
  */
 package org.apache.phoenix.end2end;
 
-import static org.apache.phoenix.end2end.ParallelStatsDisabledIT.validateQueryPlan;
 import static org.apache.phoenix.util.TestUtil.TEST_PROPERTIES;
 import static org.apache.phoenix.util.TestUtil.assertResultSet;
 import static org.junit.Assert.assertEquals;
@@ -274,15 +273,15 @@ public void testRowSizeFunction() throws Exception {
     String tableName = generateUniqueName();
     initData(conn, tableName);
 
-    // Let us establish that there are 8 rows
+    // There are 8 rows
     QueryBuilder queryBuilder =
       new QueryBuilder().setSelectExpression("count(1)").setFullTableName(tableName);
     ResultSet rs = executeQuery(conn, queryBuilder);
     assertTrue(rs.next());
     assertEquals(8, rs.getLong(1));
     assertFalse(rs.next());
 
-    // Let collect row sizes from HBase
+    // Row sizes from HBase
     int[] rowSizes = new int[8];
     int rowIndex = 0;
     ConnectionQueryServices cqs = conn.unwrap(PhoenixConnection.class).getQueryServices();
@@ -300,7 +299,7 @@ public void testRowSizeFunction() throws Exception {
       }
     }
 
-    // Verify that each row sizes is computed correctly
+    // Verify that each row sizes is computed correctly by the function row_size()
     queryBuilder = new QueryBuilder().setSelectExpression("sum(row_size())")
       .setFullTableName(tableName).setGroupByClause("ID");
     rs = executeQuery(conn, queryBuilder);
@@ -316,21 +315,22 @@ public void testRowSizeFunction() throws Exception {
       totalRowSize += rowSizes[i];
     }
 
-    // Verify that the sum function over row sizes return the expected total
+    // Verify that the sum function over row sizes returns the expected total
     queryBuilder =
       new QueryBuilder().setSelectExpression("sum(row_size())").setFullTableName(tableName);
     rs = executeQuery(conn, queryBuilder);
     assertTrue(rs.next());
     assertEquals(totalRowSize, rs.getLong(1));
     assertFalse(rs.next());
 
-    // Verify that some other aggregation functions works with row_size()
+    // Verify that some other aggregation functions work with row_size()
     queryBuilder =
       new QueryBuilder().setSelectExpression("avg(row_size()), min(row_size()), max(row_size())")
         .setFullTableName(tableName);
     rs = executeQuery(conn, queryBuilder);
     assertTrue(rs.next());
     assertEquals(totalRowSize / 8, rs.getLong(1));
+
     // There are four 90 byte rows and four 92 byte rows
     assertEquals(90, rs.getLong(2));
     assertEquals(92, rs.getLong(3));