Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5325: Do not update totalHdfsBytes_/numHdfsFiles_ on Catalogd ......................................................................
IMPALA-5325: Do not update totalHdfsBytes_/numHdfsFiles_ on Catalogd We need not account these hdfs table metrics on the Catalog server as they are eventually calculated again on the Impalads while unpacking the corresponding thrift table. This fix can potentially affect the frontend tests that directly load the Catalog's version of HdfsTable without the loadFromThrift() call paths that do the accounting. That is fixed by adding a separate call that computes these values and is called from ImpaladTestCatalog.getTable(). Testing: Enough tests already cover these code paths like show stats/ table or partition refresh tests etc. No new tests are added. Change-Id: I03cc6f9e9b2c03cafb87029ea0802dfdb2745be1 Reviewed-on: http://gerrit.cloudera.org:8080/6970 Reviewed-by: Bharath Vissapragada <[email protected]> Tested-by: Impala Public Jenkins --- M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java M fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java 2 files changed, 27 insertions(+), 13 deletions(-) Approvals: Impala Public Jenkins: Verified Bharath Vissapragada: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/6970 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I03cc6f9e9b2c03cafb87029ea0802dfdb2745be1 Gerrit-PatchSet: 9 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Impala Public Jenkins
