[kudu] branch master updated: [catalog_manager] add trace for HMS catalog timings

[alexey]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new db3a05c  [catalog_manager] add trace for HMS catalog timings
db3a05c is described below

commit db3a05c5177e241ac761daa782ca7ded914bea3f
Author: Alexey Serbin <ale...@apache.org>
AuthorDate: Mon May 6 11:03:30 2019 -0700

    [catalog_manager] add trace for HMS catalog timings
    
    Added a separate trace record for the creation of a new table
    in HMS catalog if the integration with HMS is enabled.  This helps
    to differentiate HMS-related timings from the timings of writing
    the information on the newly created table and its tables
    into the system table.
    
    Change-Id: I4946c12fdccf3a661f1dfb9a6baac4428df3d209
    Reviewed-on: http://gerrit.cloudera.org:8080/13247
    Tested-by: Alexey Serbin <aser...@cloudera.com>
    Reviewed-by: Adar Dembo <a...@cloudera.com>
---
 src/kudu/master/catalog_manager.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/kudu/master/catalog_manager.cc 
b/src/kudu/master/catalog_manager.cc
index 4a87f46..6d0e27d 100644
--- a/src/kudu/master/catalog_manager.cc
+++ b/src/kudu/master/catalog_manager.cc
@@ -1653,6 +1653,7 @@ Status CatalogManager::CreateTable(const 
CreateTableRequestPB* orig_req,
       LOG(WARNING) << s.ToString();
       return SetupError(std::move(s), resp, 
MasterErrorPB::HIVE_METASTORE_ERROR);
     }
+    TRACE("Created new table in HMS catalog");
   }
   // Delete the new HMS entry if we exit early.
   auto abort_hms = MakeScopedCleanup([&] {