Noemi Pap-Takacs has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23994 )
Change subject: IMPALA-14706: Use InMemoryMetricsReporter from the Iceberg library ...................................................................... IMPALA-14706: Use InMemoryMetricsReporter from the Iceberg library This change uses the InMemoryMetricsReporter class that is available in the Iceberg library from release 1.4.0, instead of the user declared InMemoryMetrcsReporter class. We are dropping the private static InMemoryMetricsReporter class from IcebergScanPlanner and will use the Iceberg's builtin class instead. In addition, the scanMetricsResult() method can be dropped and the relevant method scanMetrics() can be used directly. Testing: - Ran tests related to iceberg locally and examined Change-Id: I2a5acb54ad6d4b62ff6da26f250e83068a33d316 Reviewed-on: http://gerrit.cloudera.org:8080/23994 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Noemi Pap-Takacs <[email protected]> --- M fe/src/main/java/org/apache/impala/planner/IcebergScanPlanner.java 1 file changed, 14 insertions(+), 23 deletions(-) Approvals: Impala Public Jenkins: Verified Noemi Pap-Takacs: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23994 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2a5acb54ad6d4b62ff6da26f250e83068a33d316 Gerrit-Change-Number: 23994 Gerrit-PatchSet: 6 Gerrit-Owner: Surya Hebbar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
