Quanlong Huang has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17415


Change subject: IMPALA-7844: Default to not allow ordinals in HAVING clause
......................................................................

IMPALA-7844: Default to not allow ordinals in HAVING clause

Base on the discussion of the previous patch, we decide to not allow
ordinals in the HAVING clause since 4.0. It's a non-standard feature
that unintendedly supported by Impala 3.x and earlier versions.

This patch disables it by default, and add a feature flag to turn it on
for users that do depend on it.

Tests:
 - Modify existing FE tests to test on the flag.
 - Add custom cluster test to verify the flag works.

Change-Id: I0a57b8b65b046fae483e485e8391f8222fa586a5
---
M be/src/util/backend-gflag-util.cc
M common/thrift/BackendGflags.thrift
M fe/src/main/java/org/apache/impala/analysis/SelectStmt.java
M fe/src/main/java/org/apache/impala/service/BackendConfig.java
M fe/src/test/java/org/apache/impala/analysis/AnalyzeStmtsTest.java
M tests/custom_cluster/test_disable_features.py
6 files changed, 40 insertions(+), 39 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/15/17415/1
--
To view, visit http://gerrit.cloudera.org:8080/17415
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a57b8b65b046fae483e485e8391f8222fa586a5
Gerrit-Change-Number: 17415
Gerrit-PatchSet: 1
Gerrit-Owner: Quanlong Huang <[email protected]>

Reply via email to