Yida Wu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/19214
Change subject: WIP IMPALA-7969: Always admit trivial queries immediately ...................................................................... WIP IMPALA-7969: Always admit trivial queries immediately This patch adds support for the trivial query detection and allows an immediate admission for the trivial query. The definition of a trivial query: - Must have PLAN ROOT SINK as the root - Can contain UNION and EMPTYSET nodes only Examples of a trivial query: - select 1; - select * from table limit 0; TODO: Need some thinking on the definition of tirvial query - Any other cases can be trivial queries? - Any case that we don't want but allows for immediate admission in this patch? Tests: Passed exhaustive tests. Added test_trivial_query and test_trivial_query_low_mem. Change-Id: I2a729764e3055d7eb11900c96c82ff53eb261f91 --- M be/src/scheduling/admission-controller.cc M be/src/scheduling/schedule-state.cc M be/src/scheduling/schedule-state.h M be/src/service/query-options.cc M be/src/service/query-options.h M common/thrift/ImpalaService.thrift M common/thrift/Query.thrift M fe/src/main/java/org/apache/impala/planner/Planner.java A fe/src/main/java/org/apache/impala/planner/TrivialQueryChecker.java M tests/common/resource_pool_config.py M tests/custom_cluster/test_admission_controller.py M tests/custom_cluster/test_session_expiration.py M tests/custom_cluster/test_shell_interactive.py 13 files changed, 204 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/14/19214/1 -- To view, visit http://gerrit.cloudera.org:8080/19214 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I2a729764e3055d7eb11900c96c82ff53eb261f91 Gerrit-Change-Number: 19214 Gerrit-PatchSet: 1 Gerrit-Owner: Yida Wu <[email protected]>
