You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by mm...@apache.org on 2022/04/25 17:39:22 UTC

[accumulo] branch main updated: Make rest endpoints sorted (#2649)

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

mmiller pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new ab5bb96bd1 Make rest endpoints sorted (#2649)
ab5bb96bd1 is described below

commit ab5bb96bd1258b5a6720f22e855019f403dd7f1c
Author: Mike Miller <mm...@apache.org>
AuthorDate: Mon Apr 25 13:39:17 2022 -0400

    Make rest endpoints sorted (#2649)
---
 .../src/main/java/org/apache/accumulo/monitor/view/WebViews.java   | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/server/monitor/src/main/java/org/apache/accumulo/monitor/view/WebViews.java b/server/monitor/src/main/java/org/apache/accumulo/monitor/view/WebViews.java
index 4c1a981bb6..73b9732fd7 100644
--- a/server/monitor/src/main/java/org/apache/accumulo/monitor/view/WebViews.java
+++ b/server/monitor/src/main/java/org/apache/accumulo/monitor/view/WebViews.java
@@ -30,6 +30,8 @@ import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
+import java.util.TreeSet;
 
 import jakarta.inject.Inject;
 import jakarta.validation.constraints.Max;
@@ -463,8 +465,8 @@ public class WebViews {
     return model;
   }
 
-  private List<String> getEndpoints() {
-    List<String> endpoints = new ArrayList<>();
+  private Set<String> getEndpoints() {
+    Set<String> endpoints = new TreeSet<>();
     endpoints.add("/rest/manager");
     endpoints.add("/rest/tables/namespaces");
     endpoints.add("/rest/problems/summary");
@@ -488,6 +490,7 @@ public class WebViews {
     endpoints.add("/rest/statistics/time/indexCacheHitRate");
     endpoints.add("/rest/statistics/time/dataCacheHitRate");
     endpoints.add("/rest/status");
+
     return endpoints;
   }