Hello Tianyi Wang, anujphadke, Dan Hecht, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8330 to look at the new patch set (#3). Change subject: IMPALA-6080: clean up table descriptor handling ...................................................................... IMPALA-6080: clean up table descriptor handling * Add DescriptorTbl::CreateHdfsTableDescriptor to avoid having to create an entire DescriptorTbl during INSERT finalization (when only a descriptor for the output table is needed) * Remove TQueryExecRequest.desc_tbl, there's already a home for it in TQueryContext.desc_tbl This required fixing a problem in the planner test infrastructure where the TQueryCtx was reused for planning multiple times despite being modified during planning. This is based on Marcel Kornacker's coordinator cleanup patch. Testing: Ran core tests. Change-Id: Id427dab0c196b556bd8b2d64ec618403d5cbd4d6 --- M be/src/runtime/coordinator.cc M be/src/runtime/descriptors.cc M be/src/runtime/descriptors.h M common/thrift/Frontend.thrift M fe/src/main/java/org/apache/impala/service/Frontend.java M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java 6 files changed, 116 insertions(+), 110 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/30/8330/3 -- To view, visit http://gerrit.cloudera.org:8080/8330 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id427dab0c196b556bd8b2d64ec618403d5cbd4d6 Gerrit-Change-Number: 8330 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com> Gerrit-Reviewer: anujphadke <apha...@cloudera.com>