Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21059 )
Change subject: IMPALA-12426: QueryStateRecord Refactor ...................................................................... IMPALA-12426: QueryStateRecord Refactor The QueryStateRecord struct is used to store important information about a completed query for the Impala web UI page of recently completed queries. Since significant portions of this struct has data that is also needed in workload management, it has been refactored. The QueryStateRecord struct was a private child struct under the ImpalaServer class. It has now been moved to a top-level struct within the impala namespace. A new struct named QueryStateExpanded has also been created. This struct contains a shared pointer to a QueryStateRecord so the same QueryStateRecord instance can be used by both the web UI and workload management. The QueryStateExpanded struct also contains additional data that is used exclusively by workload management. New ctests have been added to exercise the added comparators. Change-Id: I57d470db6fea71ec12e43f86e3fd62ed6259c83a Reviewed-on: http://gerrit.cloudera.org:8080/21059 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/runtime/coordinator.cc M be/src/runtime/coordinator.h M be/src/service/CMakeLists.txt M be/src/service/client-request-state.cc M be/src/service/client-request-state.h M be/src/service/impala-http-handler.cc M be/src/service/impala-http-handler.h M be/src/service/impala-server.cc M be/src/service/impala-server.h A be/src/service/query-state-record-test.cc A be/src/service/query-state-record.cc A be/src/service/query-state-record.h M be/src/util/runtime-profile-test.cc M be/src/util/runtime-profile.cc M be/src/util/runtime-profile.h M common/thrift/Query.thrift M fe/src/main/java/org/apache/impala/planner/Planner.java 17 files changed, 979 insertions(+), 294 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21059 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I57d470db6fea71ec12e43f86e3fd62ed6259c83a Gerrit-Change-Number: 21059 Gerrit-PatchSet: 16 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
