fixup misc typos in comments

abhishekrb19 · abhishekrb19 · commit d3adbf562be2 · 2023-11-20T11:09:01.000-08:00
diff --git a/integration-tests/src/main/java/org/apache/druid/testing/utils/MsqTestQueryHelper.java b/integration-tests/src/main/java/org/apache/druid/testing/utils/MsqTestQueryHelper.java
@@ -130,9 +130,11 @@ public SqlTaskStatus submitMsqTask(SqlQuery sqlQuery) throws ExecutionException,
     HttpResponseStatus httpResponseStatus = statusResponseHolder.getStatus();
     if (!httpResponseStatus.equals(HttpResponseStatus.ACCEPTED)) {
       throw new ISE(
-          "Unable to submit the task successfully. Received response status code [%d], and response content:\n[%s]",
-          httpResponseStatus,
-          statusResponseHolder.getContent()
+          StringUtils.format(
+              "Unable to submit the task successfully. Received response status code [%d], and response content:\n[%s]",
+              httpResponseStatus.getCode(),
+              statusResponseHolder.getContent()
+          )
       );
     }
     String content = statusResponseHolder.getContent();
diff --git a/processing/src/main/java/org/apache/druid/timeline/partition/OverwriteShardSpec.java b/processing/src/main/java/org/apache/druid/timeline/partition/OverwriteShardSpec.java
@@ -30,7 +30,7 @@ public interface OverwriteShardSpec extends ShardSpec
 {
   /**
    * The core partition concept is not used with segment locking. Instead, the {@link AtomicUpdateGroup} is used
-   * to atomically overshadow segments. Here, we always returns 0 so that the {@link PartitionHolder} skips checking
+   * to atomically overshadow segments. Here, we always return 0 so that the {@link PartitionHolder} skips checking
    * the completeness of the core partitions.
    */
   @Override
diff --git a/server/src/main/java/org/apache/druid/server/coordinator/duty/CompactSegments.java b/server/src/main/java/org/apache/druid/server/coordinator/duty/CompactSegments.java
@@ -373,7 +373,7 @@ private int submitCompactionTasks(
 
       final String dataSourceName = segmentsToCompact.get(0).getDataSource();
 
-      // As these segments will be compacted, we will aggregates the statistic to the Compacted statistics
+      // As these segments will be compacted, we will aggregate the statistic to the Compacted statistics
       AutoCompactionSnapshot.Builder snapshotBuilder = currentRunAutoCompactionSnapshotBuilders.computeIfAbsent(
           dataSourceName,
           k -> new AutoCompactionSnapshot.Builder(k, AutoCompactionSnapshot.AutoCompactionScheduleStatus.RUNNING)
@@ -395,7 +395,7 @@ private int submitCompactionTasks(
       Granularity segmentGranularityToUse = null;
       if (config.getGranularitySpec() == null || config.getGranularitySpec().getSegmentGranularity() == null) {
         // Determines segmentGranularity from the segmentsToCompact
-        // Each batch of segmentToCompact from CompactionSegmentIterator will contains the same interval as
+        // Each batch of segmentToCompact from CompactionSegmentIterator will contain the same interval as
         // segmentGranularity is not set in the compaction config
         Interval interval = segmentsToCompact.get(0).getInterval();
         if (segmentsToCompact.stream().allMatch(segment -> interval.overlaps(segment.getInterval()))) {

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public interface OverwriteShardSpec extends ShardSpec`
`30`	`30`	`{`
`31`	`31`	`/**`
`32`	`32`	`* The core partition concept is not used with segment locking. Instead, the {@link AtomicUpdateGroup} is used`
`33`		`- * to atomically overshadow segments. Here, we always returns 0 so that the {@link PartitionHolder} skips checking`
	`33`	`+ * to atomically overshadow segments. Here, we always return 0 so that the {@link PartitionHolder} skips checking`
`34`	`34`	`* the completeness of the core partitions.`
`35`	`35`	`*/`
`36`	`36`	`@Override`