Update metric system with cherry-pick Remove hostname from metrics key

### What changes are proposed in this pull request?
cherry-pick #18121 

### Why are the changes needed?

Please clarify why the changes are needed. For instance,
  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.

### Does this PR introduce any user facing changes?

Please list the user-facing changes introduced by your change, including
  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

			pr-link: #18626
			change-id: cid-31f20b47904c5f32c9c11c08f3155cfcdcbf4e16

Author: jja725

core/common/src/main/java/alluxio/conf/PropertyKey.java

@@ -706,6 +706,13 @@ public String toString() {
           .setScope(Scope.ALL)
           .setConsistencyCheckLevel(ConsistencyCheckLevel.IGNORE)
           .build();
+  public static final PropertyKey METRICS_KEY_INCLUDING_UNIQUE_ID_ENABLED =
+      booleanBuilder(Name.METRICS_KEY_INCLUDING_UNIQUE_ID_ENABLED)
+          .setDefaultValue(false)
+          .setDescription("Whether to include the unique id such as hostname in the metrcis key.")
+          .setConsistencyCheckLevel(ConsistencyCheckLevel.WARN)
+          .setScope(Scope.ALL)
+          .build();
   public static final PropertyKey NETWORK_CONNECTION_AUTH_TIMEOUT =
       durationBuilder(Name.NETWORK_CONNECTION_AUTH_TIMEOUT)
           .setDefaultValue("30sec")
@@ -7832,6 +7839,8 @@ public static final class Name {
         "alluxio.metrics.executor.task.warn.size";
     public static final String METRICS_EXECUTOR_TASK_WARN_FREQUENCY =
         "alluxio.metrics.executor.task.warn.frequency";
+    public static final String METRICS_KEY_INCLUDING_UNIQUE_ID_ENABLED =
+        "alluxio.metrics.key.including.unique.id.enabled";
     public static final String NETWORK_CONNECTION_AUTH_TIMEOUT =
         "alluxio.network.connection.auth.timeout";
     public static final String NETWORK_CONNECTION_HEALTH_CHECK_TIMEOUT =

core/common/src/main/java/alluxio/metrics/MetricsSystem.java

@@ -74,6 +74,8 @@ public final class MetricsSystem {
   private static final ConcurrentHashMap<String, String> CACHED_METRICS = new ConcurrentHashMap<>();
   private static int sResolveTimeout =
       (int) Configuration.getMs(PropertyKey.NETWORK_HOST_RESOLUTION_TIMEOUT_MS);
+  private static boolean sUniqueIDEnabled =
+      Configuration.getBoolean(PropertyKey.METRICS_KEY_INCLUDING_UNIQUE_ID_ENABLED);
   // A map from AlluxioURI to corresponding cached escaped path.
   private static final ConcurrentHashMap<AlluxioURI, String> CACHED_ESCAPED_PATH
       = new ConcurrentHashMap<>();
@@ -482,10 +484,18 @@ public static String getPluginMetricName(String name) {
    * @return the metric registry name
    */
   private static String getMetricNameWithUniqueId(InstanceType instance, String name) {
+    String metricsNameWithInstanceType = addInstanceTypeToMetricsName(instance, name);
+    if (sUniqueIDEnabled) {
+      return Joiner.on(".").join(metricsNameWithInstanceType, sSourceNameSupplier.get());
+    }
+    return metricsNameWithInstanceType;
+  }
+
+  private static String addInstanceTypeToMetricsName(InstanceType instance, String name) {
     if (name.startsWith(instance.toString())) {
-      return Joiner.on(".").join(name, sSourceNameSupplier.get());
+      return name;
     }
-    return Joiner.on(".").join(instance, name, sSourceNameSupplier.get());
+    return Joiner.on(".").join(instance, name);
   }
 
   /**

core/common/src/test/java/alluxio/metrics/MetricsSystemTest.java

@@ -201,10 +201,10 @@ public void getMetricNameTest() {
     assertEquals("Cluster.counter", MetricsSystem.getMetricName("Cluster.counter"));
     assertEquals("Master.timer", MetricsSystem.getMetricName("Master.timer"));
     String workerGaugeName = "Worker.gauge";
-    assertNotEquals(workerGaugeName, MetricsSystem.getMetricName(workerGaugeName));
+    assertEquals(workerGaugeName, MetricsSystem.getMetricName(workerGaugeName));
     assertTrue(MetricsSystem.getMetricName(workerGaugeName).startsWith(workerGaugeName));
     String clientCounterName = "Client.counter";
-    assertNotEquals(clientCounterName, MetricsSystem.getMetricName(clientCounterName));
+    assertEquals(clientCounterName, MetricsSystem.getMetricName(clientCounterName));
     assertTrue(MetricsSystem.getMetricName(clientCounterName).startsWith(clientCounterName));
   }