Github user JamesRTaylor commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/281#discussion_r151015089
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/execute/AggregatePlan.java ---
    @@ -112,7 +114,26 @@ private AggregatePlan(StatementContext context, 
FilterableStatement statement, T
         public Expression getHaving() {
             return having;
         }
    -    
    +
    +    @Override
    +    public Cost getCost() throws SQLException {
    +        Long byteCount = getEstimatedBytesToScan();
    +        if (byteCount == null) {
    +            return Cost.ZERO;
    +        }
    +
    --- End diff --
    
    What does parallelLevel represent? Is that the level of parallelization 
done on the client? Is that derivable from thread pool size or cluster size? 
For now, maybe have a constant and doc it a bit.


---

Reply via email to