You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2026/06/18 21:20:53 UTC

(pinot) branch master updated: Fix typos in comments, Javadoc, and log messages (#18802)

This is an automated email from the ASF dual-hosted git repository.

Jackie-Jiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new f76e05afc10 Fix typos in comments, Javadoc, and log messages (#18802)
f76e05afc10 is described below

commit f76e05afc108e57076a1a7c5f9e070b864381963
Author: Akanksha kedia <89...@users.noreply.github.com>
AuthorDate: Fri Jun 19 02:50:47 2026 +0530

    Fix typos in comments, Javadoc, and log messages (#18802)
---
 .../pinot/controller/validation/RealtimeSegmentValidationManager.java   | 2 +-
 .../dropwizard/prometheus/DropwizardBrokerPrometheusMetricsTest.java    | 2 +-
 .../prometheus/DropwizardControllerPrometheusMetricsTest.java           | 2 +-
 .../dropwizard/prometheus/DropwizardMinionPrometheusMetricsTest.java    | 2 +-
 .../dropwizard/prometheus/DropwizardServerPrometheusMetricsTest.java    | 2 +-
 .../src/main/java/org/apache/pinot/spi/env/PinotConfiguration.java      | 2 +-
 .../test/java/org/apache/pinot/spi/env/VersionedPropertyConfigTest.java | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/validation/RealtimeSegmentValidationManager.java b/pinot-controller/src/main/java/org/apache/pinot/controller/validation/RealtimeSegmentValidationManager.java
index e5bb5a871bb..7f7b70821a4 100644
--- a/pinot-controller/src/main/java/org/apache/pinot/controller/validation/RealtimeSegmentValidationManager.java
+++ b/pinot-controller/src/main/java/org/apache/pinot/controller/validation/RealtimeSegmentValidationManager.java
@@ -190,7 +190,7 @@ public class RealtimeSegmentValidationManager extends ControllerPeriodicTask<Rea
         && pauseStatus.getReasonCode().equals(PauseState.ReasonCode.RESOURCE_UTILIZATION_LIMIT_EXCEEDED)) {
       // The table was previously paused due to exceeding resource utilization, but the current status cannot be
       // determined. To be safe, leave it as paused and once the status is available take the correct action
-      LOGGER.warn("Resource utilization limit could not be determined for for table: {}, and it is paused, leave it as "
+      LOGGER.warn("Resource utilization limit could not be determined for table: {}, and it is paused, leave it as "
           + "paused", tableNameWithType);
       return false;
     }
diff --git a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardBrokerPrometheusMetricsTest.java b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardBrokerPrometheusMetricsTest.java
index 619d9b7bbba..890827668c8 100644
--- a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardBrokerPrometheusMetricsTest.java
+++ b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardBrokerPrometheusMetricsTest.java
@@ -31,7 +31,7 @@ import org.testng.annotations.Test;
 
 
 /**
- * Disabling tests as Pinot currently uses Yammer and these tests fail for for {@link DropwizardMetricsFactory}
+ * Disabling tests as Pinot currently uses Yammer and these tests fail for {@link DropwizardMetricsFactory}
  */
 @Test(enabled = false) // enabled=false on class level doesn't seem to work in intellij
 public class DropwizardBrokerPrometheusMetricsTest extends BrokerPrometheusMetricsTest {
diff --git a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardControllerPrometheusMetricsTest.java b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardControllerPrometheusMetricsTest.java
index 0ae21cf9339..61a288f3f89 100644
--- a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardControllerPrometheusMetricsTest.java
+++ b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardControllerPrometheusMetricsTest.java
@@ -29,7 +29,7 @@ import org.testng.annotations.Test;
 
 
 /**
- * Disabling tests as Pinot currently uses Yammer and these tests fail for for {@link DropwizardMetricsFactory}
+ * Disabling tests as Pinot currently uses Yammer and these tests fail for {@link DropwizardMetricsFactory}
  */
 @Test(enabled = false) // enabled=false on class level doesn't seem to work in intellij
 public class DropwizardControllerPrometheusMetricsTest extends ControllerPrometheusMetricsTest {
diff --git a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardMinionPrometheusMetricsTest.java b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardMinionPrometheusMetricsTest.java
index ae79ffa9bf1..d281e7ff99d 100644
--- a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardMinionPrometheusMetricsTest.java
+++ b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardMinionPrometheusMetricsTest.java
@@ -29,7 +29,7 @@ import org.testng.annotations.Test;
 
 
 /**
- * Disabling tests as Pinot currently uses Yammer and these tests fail for for {@link DropwizardMetricsFactory}
+ * Disabling tests as Pinot currently uses Yammer and these tests fail for {@link DropwizardMetricsFactory}
  */
 @Test(enabled = false) // enabled=false on class level doesn't seem to work in intellij
 public class DropwizardMinionPrometheusMetricsTest extends MinionPrometheusMetricsTest {
diff --git a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardServerPrometheusMetricsTest.java b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardServerPrometheusMetricsTest.java
index 13be096fc28..ec52900c3b3 100644
--- a/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardServerPrometheusMetricsTest.java
+++ b/pinot-plugins/pinot-metrics/pinot-dropwizard/src/test/java/org/apache/pinot/plugin/metrics/dropwizard/prometheus/DropwizardServerPrometheusMetricsTest.java
@@ -31,7 +31,7 @@ import org.testng.annotations.Test;
 
 
 /**
- * Disabling tests as Pinot currently uses Yammer and these tests fail for for {@link DropwizardMetricsFactory}
+ * Disabling tests as Pinot currently uses Yammer and these tests fail for {@link DropwizardMetricsFactory}
  */
 @Test(enabled = false) // enabled=false on class level doesn't seem to work in intellij
 public class DropwizardServerPrometheusMetricsTest extends ServerPrometheusMetricsTest {
diff --git a/pinot-spi/src/main/java/org/apache/pinot/spi/env/PinotConfiguration.java b/pinot-spi/src/main/java/org/apache/pinot/spi/env/PinotConfiguration.java
index 334896b0432..7ab7837fdad 100644
--- a/pinot-spi/src/main/java/org/apache/pinot/spi/env/PinotConfiguration.java
+++ b/pinot-spi/src/main/java/org/apache/pinot/spi/env/PinotConfiguration.java
@@ -43,7 +43,7 @@ import org.slf4j.LoggerFactory;
  * Provides a configuration abstraction for Pinot to decouple services from configuration sources and frameworks.
  * </p>
  * <p>
- * Pinot services may retreived configurations from PinotConfiguration independently from any source of configuration.
+ * Pinot services may retrieve configurations from PinotConfiguration independently from any source of configuration.
  * {@link PinotConfiguration} currently supports configuration loaded from the following sources :
  *
  * <ul>
diff --git a/pinot-spi/src/test/java/org/apache/pinot/spi/env/VersionedPropertyConfigTest.java b/pinot-spi/src/test/java/org/apache/pinot/spi/env/VersionedPropertyConfigTest.java
index 823adff2a6c..b31df4b8986 100644
--- a/pinot-spi/src/test/java/org/apache/pinot/spi/env/VersionedPropertyConfigTest.java
+++ b/pinot-spi/src/test/java/org/apache/pinot/spi/env/VersionedPropertyConfigTest.java
@@ -107,7 +107,7 @@ public class VersionedPropertyConfigTest {
 
   @Test
   //Test requires 'segment-metadata-without-version-header.properties' sample segment metadata file in resources folder
-  public void testOldSegmentMetadataBackwardCompatability()
+  public void testOldSegmentMetadataBackwardCompatibility()
       throws ConfigurationException {
     File oldSegmentProperties = new File(
         Objects.requireNonNull(


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org