Hello Philip Zeyliger, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11920
to look at the new patch set (#2).
Change subject: IMPALA-7842: Make query fragments available for unit testing
......................................................................
IMPALA-7842: Make query fragments available for unit testing
Today, the FrontEnd class uses a functional model to generate a plan:
pass in SQL text to the createExecRequest() method and get back a Thrift
plan ready for serialization.
For unit testing, however, we need access to the intermediate plan
fragments produced by the planner. Inspecting only the Thrift version
results in loss of important details.
This fix introduces a new planner context (PlanCtx) class that passes
information into the planner, and can pass the plan fragments back out.
There is no functional change, just a refactoring of the existing code.
The changes also introduces a new test case that uses this feature to
verify plan cardinality.
Tests: The new test case demonstrates the functionality.
Change-Id: I4c615dbc1d14685a6966c5ca7538777cdc80b74d
---
M fe/src/main/java/org/apache/impala/service/Frontend.java
A fe/src/test/java/org/apache/impala/planner/CardinalityTest.java
2 files changed, 219 insertions(+), 16 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/20/11920/2
--
To view, visit http://gerrit.cloudera.org:8080/11920
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4c615dbc1d14685a6966c5ca7538777cdc80b74d
Gerrit-Change-Number: 11920
Gerrit-PatchSet: 2
Gerrit-Owner: Paul Rogers <par0...@yahoo.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Paul Rogers <par0...@yahoo.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
