Riza Suminto has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/20027


Change subject: IMPALA-12196: Handle empty REQUEST_POOL in Frontend.java
......................................................................

IMPALA-12196: Handle empty REQUEST_POOL in Frontend.java

When using impala-shell client, SET REQUEST_POOL="" does not actually
unset the query option. Instead, it pass an empty string that may fail
executor group checking in
Frontend.setupThresholdsForExecutorGroupSets(). This patch fix the issue
by replacing the null checking with StringUtils.isNotEmpty() that
returns True if the given parameter is null or empty string.

Testing:
- Manually test the scenario mentioned in IMPALA-12196 ticket with
  impala-shell and query success without hitting AnalysisException.

Change-Id: Ibc3495ac117f08f1165e9e5e9a70856cbee7b0ef
---
M fe/src/main/java/org/apache/impala/service/Frontend.java
1 file changed, 5 insertions(+), 2 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibc3495ac117f08f1165e9e5e9a70856cbee7b0ef
Gerrit-Change-Number: 20027
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <[email protected]>

Reply via email to