Riza Suminto has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20684
Change subject: IMPALA-12548: Clarify the meaning of AGG_MEM_CORRELATION_FACTOR ...................................................................... IMPALA-12548: Clarify the meaning of AGG_MEM_CORRELATION_FACTOR AGG_MEM_CORRELATION_FACTOR has valid value between 0.0 to 1.0. This query option name can be misinterpreted as high value means highly correlated columns, but it is actually the opposite in the code implementation. 1.0 is interpreted as no correlation at all in the code, while <1.0 means somewhat correlated. This patch add such clarification in code and thrift comments. Change-Id: I6f81db32a1818abc257957f6de942b5c9f36211a --- M common/thrift/ImpalaService.thrift M fe/src/main/java/org/apache/impala/planner/AggregationNode.java 2 files changed, 12 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/84/20684/1 -- To view, visit http://gerrit.cloudera.org:8080/20684 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I6f81db32a1818abc257957f6de942b5c9f36211a Gerrit-Change-Number: 20684 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <[email protected]>
