Add metrics for get table parameters (Netflix#555)

* Add metrics for get table parameters * Fix javadoc * Checkstyle * Move get table metrics to request wrapper * Remove hardcoded tags for catalog and database
yaohwu · May 12, 2023 · 1c26d5d · 1c26d5d
1 parent 33cb4be
commit 1c26d5d
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/metacat-main/src/main/java/com/netflix/metacat/main/api/v1/MetacatController.java b/metacat-main/src/main/java/com/netflix/metacat/main/api/v1/MetacatController.java
@@ -14,6 +14,7 @@
 package com.netflix.metacat.main.api.v1;
 
 import com.google.common.base.Preconditions;
+import com.google.common.collect.ImmutableMap;
 import com.netflix.metacat.common.NameDateDto;
 import com.netflix.metacat.common.QualifiedName;
 import com.netflix.metacat.common.dto.CatalogDto;
@@ -686,6 +687,13 @@ public TableDto getTable(
         return this.requestWrapper.processRequest(
                 name,
                 "getTable",
+                ImmutableMap.<String, String>builder()
+                    .put("includeInfo", String.valueOf(includeInfo))
+                    .put("includeDefinitionMetadata", String.valueOf(includeDefinitionMetadata))
+                    .put("includeDataMetadata", String.valueOf(includeDataMetadata))
+                    .put("includeMetadataFromConnector", String.valueOf(includeInfoDetails))
+                    .put("includeMetadataLocationOnly", String.valueOf(includeMetadataLocationOnly))
+                    .build(),
                 () -> {
                     final Optional<TableDto> table = this.tableService.get(
                             name,