Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8148 )
Change subject: IMPALA-4252: Move runtime filters to ScanNode ...................................................................... IMPALA-4252: Move runtime filters to ScanNode As a preliminary step towards adding runtime filters for Kudu scans, this patch moves runtime filter related code from HdfsScanNodeBase to ScanNode so that it's available to KuduScanNodeBase. The change was mechanical with no logic changes, except for moving the calculation of the max wait time into WaitForRuntimeFilters(). Testing: - Ran existing runtime filter tests. Change-Id: I17bdc869046dc2cd837d02f333441fa6324ff086 Reviewed-on: http://gerrit.cloudera.org:8080/8148 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/exec/hdfs-scan-node-base.cc M be/src/exec/hdfs-scan-node-base.h M be/src/exec/kudu-scan-node-base.cc M be/src/exec/kudu-scan-node-base.h M be/src/exec/scan-node.cc M be/src/exec/scan-node.h 6 files changed, 131 insertions(+), 114 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8148 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I17bdc869046dc2cd837d02f333441fa6324ff086 Gerrit-Change-Number: 8148 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
