IMPALA-6516: Log catalog update only if the catalog version changes Impalad writes a line of log whenever a statestore catalog update comes in. This patch removes the logging when the catalog version doesn't change.
Change-Id: I04b8dd05c588d4cd91e9ca2251f8f66325bb45e2 Reviewed-on: http://gerrit.cloudera.org:8080/9311 Reviewed-by: Michael Ho <k...@cloudera.com> Reviewed-by: anujphadke <apha...@cloudera.com> Reviewed-by: Alex Behm <alex.b...@cloudera.com> Tested-by: Impala Public Jenkins Project: http://git-wip-us.apache.org/repos/asf/impala/repo Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/d45de474 Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/d45de474 Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/d45de474 Branch: refs/heads/master Commit: d45de4747c57fdfb6b88f30a4b4b7685696277fc Parents: d7f2ce1 Author: Tianyi Wang <tw...@cloudera.com> Authored: Tue Feb 13 13:21:35 2018 -0800 Committer: Impala Public Jenkins <impala-public-jenk...@gerrit.cloudera.org> Committed: Wed Feb 14 05:26:48 2018 +0000 ---------------------------------------------------------------------- be/src/service/impala-server.cc | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/impala/blob/d45de474/be/src/service/impala-server.cc ---------------------------------------------------------------------- diff --git a/be/src/service/impala-server.cc b/be/src/service/impala-server.cc index 9898029..d5be4dc 100644 --- a/be/src/service/impala-server.cc +++ b/be/src/service/impala-server.cc @@ -1347,13 +1347,15 @@ void ImpalaServer::CatalogUpdateCallback( } else { { unique_lock<mutex> unique_lock(catalog_version_lock_); + if (catalog_update_info_.catalog_version != resp.new_catalog_version) { + LOG(INFO) << "Catalog topic update applied with version: " << + resp.new_catalog_version << " new min catalog object version: " << + resp.min_catalog_object_version; + } catalog_update_info_.catalog_version = resp.new_catalog_version; catalog_update_info_.catalog_topic_version = delta.to_version; catalog_update_info_.catalog_service_id = resp.catalog_service_id; catalog_update_info_.min_catalog_object_version = resp.min_catalog_object_version; - LOG(INFO) << "Catalog topic update applied with version: " << - resp.new_catalog_version << " new min catalog object version: " << - resp.min_catalog_object_version; catalog_update_info_.UpdateCatalogVersionMetrics(); } ImpaladMetrics::CATALOG_READY->SetValue(resp.new_catalog_version > 0);