Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21794 )
Change subject: IMPALA-13378: Verify tuple ids in descriptor table received in executor side ...................................................................... IMPALA-13378: Verify tuple ids in descriptor table received in executor side It's suspicious that KRPC could receive incomplete sidecar data (KUDU-3582). TQueryCtx and TExecPlanFragmentInfo are both sent as sidecars. Executors should check whether tuple ids in TExecPlanFragmentInfo are consistent with the descriptor table deserialized from TQueryCtx. This patch adds the check when launching the fragment instance threads. Tests - ran CORE tests Change-Id: I0c489d3bff7ae08813271c65086ea6b238420e47 Reviewed-on: http://gerrit.cloudera.org:8080/21794 Reviewed-by: Wenzhe Zhou <[email protected]> Reviewed-by: Michael Smith <[email protected]> Tested-by: Michael Smith <[email protected]> --- M be/src/runtime/query-state.cc 1 file changed, 17 insertions(+), 0 deletions(-) Approvals: Wenzhe Zhou: Looks good to me, but someone else must approve Michael Smith: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21794 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0c489d3bff7ae08813271c65086ea6b238420e47 Gerrit-Change-Number: 21794 Gerrit-PatchSet: 3 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
