isAggregationExpr seems redundant. Use "(m_aggregationExprs.size() != 0)" once to set aggExpr and use "(aggExpr)" or "(aggExpr != null)" thereafter.