Rename stat type fields

Signed-off-by: Andy Qin <[email protected]>

Author: q-andy

src/main/java/org/opensearch/neuralsearch/stats/events/EventStatName.java

@@ -18,7 +18,7 @@
  */
 @Getter
 public enum EventStatName implements StatName {
-    TEXT_EMBEDDING_PROCESSOR_EXECUTIONS("text_embedding_executions", "processors.ingest", EventStatType.TIMESTAMPED_COUNTER);
+    TEXT_EMBEDDING_PROCESSOR_EXECUTIONS("text_embedding_executions", "processors.ingest", EventStatType.TIMESTAMPED_EVENT_COUNTER);
 
     private final String nameString;
     private final String path;
@@ -43,7 +43,7 @@ public enum EventStatName implements StatName {
         this.statType = statType;
 
         switch (statType) {
-            case EventStatType.TIMESTAMPED_COUNTER:
+            case EventStatType.TIMESTAMPED_EVENT_COUNTER:
                 eventStat = new TimestampedEventStat(this);
                 break;
         }

src/main/java/org/opensearch/neuralsearch/stats/events/EventStatType.java

@@ -12,7 +12,7 @@
  * Enum for different kinds of event stat types to track
  */
 public enum EventStatType implements StatType {
-    TIMESTAMPED_COUNTER;
+    TIMESTAMPED_EVENT_COUNTER;
 
     /**
      * Gets the name of the stat type, the enum name in lowercase

src/main/java/org/opensearch/neuralsearch/stats/events/EventStatsManager.java

@@ -72,7 +72,7 @@ public Map<EventStatName, TimestampedEventStatSnapshot> getTimestampedEventStatS
         // Filter stats based on passed in collection
         Map<EventStatName, TimestampedEventStatSnapshot> eventStatsDataMap = new HashMap<>();
         for (EventStatName statName : statsToRetrieve) {
-            if (statName.getStatType() == EventStatType.TIMESTAMPED_COUNTER) {
+            if (statName.getStatType() == EventStatType.TIMESTAMPED_EVENT_COUNTER) {
                 StatSnapshot<?> snapshot = statName.getEventStat().getStatSnapshot();
                 if (snapshot instanceof TimestampedEventStatSnapshot) {
                     // Get event data snapshot

src/main/java/org/opensearch/neuralsearch/stats/info/InfoStatName.java

@@ -19,8 +19,8 @@
 @Getter
 public enum InfoStatName implements StatName {
     // Cluster info
-    CLUSTER_VERSION("cluster_version", "", InfoStatType.SETTABLE_STRING),
-    TEXT_EMBEDDING_PROCESSORS("text_embedding_processors_in_pipelines", "processors.ingest", InfoStatType.INFO_COUNTABLE);
+    CLUSTER_VERSION("cluster_version", "", InfoStatType.INFO_STRING),
+    TEXT_EMBEDDING_PROCESSORS("text_embedding_processors_in_pipelines", "processors.ingest", InfoStatType.INFO_COUNTER);
 
     private final String nameString;
     private final String path;

src/main/java/org/opensearch/neuralsearch/stats/info/InfoStatType.java

@@ -12,9 +12,9 @@
  * Enum for different kinds of info stat types to track
  */
 public enum InfoStatType implements StatType {
-    INFO_COUNTABLE,
-    SETTABLE_STRING,
-    SETTABLE_BOOLEAN;
+    INFO_COUNTER,
+    INFO_STRING,
+    INFO_BOOLEAN;
 
     /**
      * Gets the name of the stat type, the enum name in lowercase

src/main/java/org/opensearch/neuralsearch/stats/info/InfoStatsManager.java

@@ -72,7 +72,7 @@ private Map<InfoStatName, CountableInfoStatSnapshot> getCountableStats() {
         // Initialize empty map with keys so stat names are visible in JSON even if the value is not counted
         Map<InfoStatName, CountableInfoStatSnapshot> countableInfoStats = new HashMap<>();
         for (InfoStatName stat : EnumSet.allOf(InfoStatName.class)) {
-            if (stat.getStatType() == InfoStatType.INFO_COUNTABLE) {
+            if (stat.getStatType() == InfoStatType.INFO_COUNTER) {
                 countableInfoStats.put(stat, new CountableInfoStatSnapshot(stat));
             }
         }
@@ -92,8 +92,8 @@ private Map<InfoStatName, SettableInfoStatSnapshot<?>> getSettableStats() {
         Map<InfoStatName, SettableInfoStatSnapshot<?>> settableInfoStats = new HashMap<>();
         for (InfoStatName statName : EnumSet.allOf(InfoStatName.class)) {
             switch (statName.getStatType()) {
-                case InfoStatType.SETTABLE_BOOLEAN -> settableInfoStats.put(statName, new SettableInfoStatSnapshot<Boolean>(statName));
-                case InfoStatType.SETTABLE_STRING -> settableInfoStats.put(statName, new SettableInfoStatSnapshot<String>(statName));
+                case InfoStatType.INFO_BOOLEAN -> settableInfoStats.put(statName, new SettableInfoStatSnapshot<Boolean>(statName));
+                case InfoStatType.INFO_STRING -> settableInfoStats.put(statName, new SettableInfoStatSnapshot<String>(statName));
             }
         }
 

src/test/java/org/opensearch/neuralsearch/rest/RestNeuralStatsActionIT.java

@@ -138,7 +138,7 @@ public void test_includeMetadata() throws Exception {
         String valueWithMetadata = ((Map<String, String>) clusterVersionStatMetadata).get(StatSnapshot.VALUE_FIELD);
 
         // Stat type metadata should match
-        assertEquals(InfoStatType.SETTABLE_STRING.getTypeString(), statType);
+        assertEquals(InfoStatType.INFO_STRING.getTypeString(), statType);
 
         // Fetch Without metadata
         params.put(RestNeuralStatsAction.INCLUDE_METADATA_PARAM, "false");