Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/3851
Change subject: IMPALA-3873: Add QueryStateAccessor ...................................................................... IMPALA-3873: Add QueryStateAccessor In-flight and completed queries are tracked by different data structures in ImpalaServer. However, we often want to access both in the same way, by reading various bits of query metadata that are common to both. This would involve special-case logic depending on what state the query was in. This patch adds ImpalaServer::QueryStateAccessor which wraps either a QueryExecState or a QueryStateRecord and provides a common struct-like interface to both. This is mostly useful for the debug webpages that use these accessors to generate JSON for the query tables etc. Change-Id: Iec3dae66a81988c99cde1516ff511186e17dd8c0 --- M be/src/service/CMakeLists.txt M be/src/service/impala-http-handler.cc M be/src/service/impala-server.cc M be/src/service/impala-server.h A be/src/service/query-state-accessor.cc A be/src/service/query-state-accessor.h 6 files changed, 337 insertions(+), 138 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/51/3851/1 -- To view, visit http://gerrit.cloudera.org:8080/3851 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iec3dae66a81988c99cde1516ff511186e17dd8c0 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Henry Robinson <[email protected]>
