Fixing violations

tandonks · tandonks · commit 4574641df0ed · 2025-08-10T09:36:43.000+05:30
Signed-off-by: Kshitij Tandon &lt;tandonks@amazon.com&gt;
diff --git a/server/src/main/java/org/opensearch/search/aggregations/metrics/ScriptedMetricAggregator.java b/server/src/main/java/org/opensearch/search/aggregations/metrics/ScriptedMetricAggregator.java
@@ -158,8 +158,7 @@ public void collect(int doc, long owningBucketOrd) throws IOException {
     @Override
     public InternalAggregation buildAggregation(long owningBucketOrdinal) {
         Object result = aggStateForResult(owningBucketOrdinal).combine();
-        if (result.getClass() != ScriptedAvg.class)
-            StreamOutput.checkWriteable(result);
+        if (result.getClass() != ScriptedAvg.class) StreamOutput.checkWriteable(result);
         return new InternalScriptedMetric(name, singletonList(result), reduceScript, metadata());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -158,8 +158,7 @@ public void collect(int doc, long owningBucketOrd) throws IOException {`
`158`	`158`	`@Override`
`159`	`159`	`public InternalAggregation buildAggregation(long owningBucketOrdinal) {`
`160`	`160`	`Object result = aggStateForResult(owningBucketOrdinal).combine();`
`161`		`- if (result.getClass() != ScriptedAvg.class)`
`162`		`- StreamOutput.checkWriteable(result);`
	`161`	`+ if (result.getClass() != ScriptedAvg.class) StreamOutput.checkWriteable(result);`
`163`	`162`	`return new InternalScriptedMetric(name, singletonList(result), reduceScript, metadata());`
`164`	`163`	`}`
`165`	`164`