Minor fix

Signed-off-by: Siddhant Deshmukh <[email protected]>
deshsidd · Oct 16, 2023 · 4148d6a · 4148d6a
1 parent 929ef7d
commit 4148d6a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/server/src/main/java/org/opensearch/action/search/SearchQueryCategorizer.java b/server/src/main/java/org/opensearch/action/search/SearchQueryCategorizer.java
@@ -57,7 +57,6 @@ private void incrementQueryAggregationCounters(AggregatorFactories.Builder aggre
 
     private void incrementQueryTypeCounters(QueryBuilder topLevelQueryBuilder) {
         if (topLevelQueryBuilder == null) {
-            searchQueryCounters.skippedCounter.add(1);
             return;
         }
         QueryBuilderVisitor searchQueryVisitor = new SearchQueryCategorizingVisitor(searchQueryCounters);
@@ -66,7 +65,6 @@ private void incrementQueryTypeCounters(QueryBuilder topLevelQueryBuilder) {
 
     private void logQueryShape(QueryBuilder topLevelQueryBuilder) {
         if (topLevelQueryBuilder == null) {
-            searchQueryCounters.skippedCounter.add(1);
             return;
         }
         QueryShapeVisitor shapeVisitor = new QueryShapeVisitor();

diff --git a/server/src/main/java/org/opensearch/common/util/FeatureFlags.java b/server/src/main/java/org/opensearch/common/util/FeatureFlags.java
@@ -59,7 +59,7 @@ public class FeatureFlags {
      * Gates the optimization of datetime formatters caching along with change in default datetime formatter.
      */
     public static final String DATETIME_FORMATTER_CACHING = "opensearch.experimental.optimization.datetime_formatter_caching.enabled";
-    
+
     /**
      * Should store the settings from opensearch.yml.
      */
-Original file line number
+Diff line change
@@ Expand Up / @@ -59,7 +59,7 @@ public class FeatureFlags { @@
          * Gates the optimization of datetime formatters caching along with change in default datetime formatter.
          */
         public static final String DATETIME_FORMATTER_CACHING = "opensearch.experimental.optimization.datetime_formatter_caching.enabled";
         /**
          * Should store the settings from opensearch.yml.
          */
@@ Expand Down @@