Tighten inline comments

Signed-off-by: Kai Huang <ahkcs@amazon.com>

Author: ahkcs

opensearch/src/main/java/org/opensearch/sql/opensearch/request/system/OpenSearchDescribeIndexRequest.java

@@ -117,9 +117,8 @@ public Map<String, OpenSearchDataType> getFieldTypes() {
         fieldTypes.putAll(indexMapping.getFieldMappings());
       }
     } else {
-      // Merge deep copies: MergeRuleHelper rewrites the accumulated type's nested `properties` in
-      // place, which would otherwise mutate the per-index mappings retained above (they are reused
-      // by partial-result partitioning, which needs to see each index's original mapping).
+      // Merge deep copies: MergeRuleHelper mutates the field mappings in place, and the per-index
+      // mappings retained above must stay intact for partial-result partitioning.
       for (IndexMapping indexMapping : indexMappings.values()) {
         MergeRuleHelper.merge(fieldTypes, deepCopy(indexMapping.getFieldMappings()));
       }

opensearch/src/main/java/org/opensearch/sql/opensearch/storage/scan/CalciteLogicalIndexScan.java

@@ -411,10 +411,8 @@ private AbstractRelNode pushDownAggregate(
         }
         return null;
       }
-      // Decide partial mode before analyze, not after a failure: since #5646 a text/keyword
-      // conflict pushes down as a (slow) _source script rather than failing, so a post-failure
-      // fallback would never fire. Array/nested buckets (above) and other analyze failures are not
-      // text/keyword conflicts, so partial mode does not apply to them.
+      // Try partial mode before analyze: since #5646 a text/keyword conflict pushes down as a slow
+      // _source script instead of failing, so a post-failure fallback would never fire.
       if (allowPartialFallback) {
         AbstractRelNode partial = tryPartialResultAggregate(aggregate, project, bucketNames);
         if (partial != null) {

plugin/src/main/java/org/opensearch/sql/plugin/request/PPLQueryRequestFactory.java

@@ -132,8 +132,8 @@ private static PPLQueryRequest parsePPLRequestFromPayload(RestRequest restReques
       if (queryId != null) {
         pplRequest.queryId(queryId);
       }
-      // set per-request partial-result override only when explicitly present, so a request that
-      // says nothing defers to the cluster setting rather than forcing the flag off.
+      // Set the override only when present, so a request that omits it defers to the cluster
+      // setting.
       if (jsonContent.has(QUERY_PARAMS_PARTIAL_RESULT)) {
         pplRequest.partialResult(jsonContent.optBoolean(QUERY_PARAMS_PARTIAL_RESULT));
       }

plugin/src/main/java/org/opensearch/sql/plugin/transport/TransportPPLQueryAction.java

@@ -180,11 +180,10 @@ protected void doExecute(
     // in order to use PPL service, we need to convert TransportPPLQueryRequest to PPLQueryRequest
     PPLQueryRequest transformedRequest = transportRequest.toPPLQueryRequest();
     QueryContext.setProfile(transformedRequest.profile());
-    // Only the JSON response shape carries a warnings channel. Features that return a
-    // knowingly-partial result gate on this so they never silently drop data into CSV/RAW/VIZ.
+    // Only the JSON shape carries warnings; gate partial results on it so CSV/RAW/VIZ never drop
+    // data silently.
     QueryContext.setWarningsSupported(warningsSupported(transformedRequest));
-    // Per-request partial-result override (e.g. from a Dashboards toggle); null defers to the
-    // cluster setting.
+    // Per-request override (e.g. a Dashboards toggle); null defers to the cluster setting.
     QueryContext.setPartialResultOverride(transformedRequest.partialResult());
     ActionListener<TransportPPLQueryResponse> clearingListener = wrapWithProfilingClear(listener);