This is an automated email from the ASF dual-hosted git repository.
xiangfu 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 d40e41d799b Correct typos in code comments across multiple Java files
(#17093)
d40e41d799b is described below
commit d40e41d799beafa65133423130e68a783ccd4f99
Author: Akanksha kedia <[email protected]>
AuthorDate: Tue Oct 28 09:55:22 2025 +0530
Correct typos in code comments across multiple Java files (#17093)
---
.../org/apache/pinot/controller/helix/IdealStateGroupCommitTest.java | 2 +-
.../java/org/apache/pinot/controller/util/ListenerConfigUtilTest.java | 2 +-
.../pinot/core/operator/transform/function/ItemTransformFunction.java | 4 ++--
.../test/java/org/apache/pinot/queries/ExplainPlanQueriesTest.java | 4 ++--
.../main/java/org/apache/pinot/plugin/filesystem/ADLSGen2PinotFS.java | 2 +-
.../impl/FixedByteSingleValueMultiColumnReaderWriter.java | 2 +-
.../apache/pinot/segment/local/utils/nativefst/automaton/RegExp.java | 4 ++--
7 files changed, 10 insertions(+), 10 deletions(-)
diff --git
a/pinot-controller/src/test/java/org/apache/pinot/controller/helix/IdealStateGroupCommitTest.java
b/pinot-controller/src/test/java/org/apache/pinot/controller/helix/IdealStateGroupCommitTest.java
index 98c8aa6ef48..37728cacf13 100644
---
a/pinot-controller/src/test/java/org/apache/pinot/controller/helix/IdealStateGroupCommitTest.java
+++
b/pinot-controller/src/test/java/org/apache/pinot/controller/helix/IdealStateGroupCommitTest.java
@@ -117,7 +117,7 @@ public class IdealStateGroupCommitTest {
long idealStateUpdateSuccessCount =
controllerMetrics.getMeteredTableValue(tableName,
ControllerMeter.IDEAL_STATE_UPDATE_SUCCESS).count();
Assert.assertTrue(idealStateUpdateSuccessCount <= NUM_UPDATES);
- LOGGER.info("{} IdealState update are successfully commited with {}
times zk updates.", NUM_UPDATES,
+ LOGGER.info("{} IdealState update are successfully committed with {}
times zk updates.", NUM_UPDATES,
idealStateUpdateSuccessCount);
}
}
diff --git
a/pinot-controller/src/test/java/org/apache/pinot/controller/util/ListenerConfigUtilTest.java
b/pinot-controller/src/test/java/org/apache/pinot/controller/util/ListenerConfigUtilTest.java
index f88e87fb760..ca12560ac31 100644
---
a/pinot-controller/src/test/java/org/apache/pinot/controller/util/ListenerConfigUtilTest.java
+++
b/pinot-controller/src/test/java/org/apache/pinot/controller/util/ListenerConfigUtilTest.java
@@ -101,7 +101,7 @@ public class ListenerConfigUtilTest {
}
/**
- * Asserts that controller.port can be opt-out and both http and https can
be configured with seperate ports.
+ * Asserts that controller.port can be opt-out and both http and https can
be configured with separate ports.
*/
@Test
public void testHttpAndHttpsConfigs() {
diff --git
a/pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/ItemTransformFunction.java
b/pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/ItemTransformFunction.java
index cfb80760eeb..cdb5c12c9ca 100644
---
a/pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/ItemTransformFunction.java
+++
b/pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/ItemTransformFunction.java
@@ -70,8 +70,8 @@ public class ItemTransformFunction extends
BaseTransformFunction {
+ "must be a string literal");
_keyPath = new String[]{_column, _key};
- // The metadata about the values that this operation will resolve to is
determined by the type of teh data
- // under they key, not by the Map column. So we need to look up the Key's
Metadata.
+ // The metadata about the values that this operation will resolve to is
determined by the type of the data
+ // under the key, not by the Map column. So we need to look up the Key's
Metadata.
DataSource dataSource = columnContextMap.get(_column).getDataSource();
if (dataSource instanceof MapDataSource) {
diff --git
a/pinot-core/src/test/java/org/apache/pinot/queries/ExplainPlanQueriesTest.java
b/pinot-core/src/test/java/org/apache/pinot/queries/ExplainPlanQueriesTest.java
index 11f067c0161..6de2f2a1741 100644
---
a/pinot-core/src/test/java/org/apache/pinot/queries/ExplainPlanQueriesTest.java
+++
b/pinot-core/src/test/java/org/apache/pinot/queries/ExplainPlanQueriesTest.java
@@ -1786,7 +1786,7 @@ public class ExplainPlanQueriesTest extends
BaseQueriesTest {
result1.add(new Object[]{"AGGREGATE_NO_SCAN", 3, 2});
check(query1, new ResultTable(DATA_SCHEMA, result1));
- // No scan required as metadata is sufficient to answer teh query for all
segments
+ // No scan required as metadata is sufficient to answer the query for all
segments
String query2 = "EXPLAIN PLAN FOR SELECT min(invertedIndexCol1) FROM
testTable";
List<Object[]> result2 = new ArrayList<>();
result2.add(new Object[]{"BROKER_REDUCE(limit:10)", 1, 0});
@@ -1898,7 +1898,7 @@ public class ExplainPlanQueriesTest extends
BaseQueriesTest {
result1.add(new Object[]{"AGGREGATE_NO_SCAN", 3, 2});
check(query1, new ResultTable(DATA_SCHEMA, result1));
- // No scan required as metadata is sufficient to answer teh query for all
segments
+ // No scan required as metadata is sufficient to answer the query for all
segments
String query2 = "SET explainPlanVerbose=true; EXPLAIN PLAN FOR SELECT
min(invertedIndexCol1) FROM testTable";
List<Object[]> result2 = new ArrayList<>();
result2.add(new Object[]{"BROKER_REDUCE(limit:10)", 1, 0});
diff --git
a/pinot-plugins/pinot-file-system/pinot-adls/src/main/java/org/apache/pinot/plugin/filesystem/ADLSGen2PinotFS.java
b/pinot-plugins/pinot-file-system/pinot-adls/src/main/java/org/apache/pinot/plugin/filesystem/ADLSGen2PinotFS.java
index 47613387398..0f294f9103e 100644
---
a/pinot-plugins/pinot-file-system/pinot-adls/src/main/java/org/apache/pinot/plugin/filesystem/ADLSGen2PinotFS.java
+++
b/pinot-plugins/pinot-file-system/pinot-adls/src/main/java/org/apache/pinot/plugin/filesystem/ADLSGen2PinotFS.java
@@ -396,7 +396,7 @@ public class ADLSGen2PinotFS extends BasePinotFS {
/**
* Checks if the file exists at a given location
*
- * @param fileUri location to check the existance of the file.
+ * @param fileUri location to check the existence of the file.
* @return true if exists else false.
*/
@Override
diff --git
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/io/readerwriter/impl/FixedByteSingleValueMultiColumnReaderWriter.java
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/io/readerwriter/impl/FixedByteSingleValueMultiColumnReaderWriter.java
index 601b09a2510..38e3741c8cd 100644
---
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/io/readerwriter/impl/FixedByteSingleValueMultiColumnReaderWriter.java
+++
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/io/readerwriter/impl/FixedByteSingleValueMultiColumnReaderWriter.java
@@ -186,7 +186,7 @@ public class FixedByteSingleValueMultiColumnReaderWriter
implements Closeable {
FixedByteSingleValueMultiColReader reader =
new FixedByteSingleValueMultiColReader(buffer, _numRowsPerChunk,
_columnSizesInBytes);
- // ArrayList is non-threadsafe. So add to a new copy and then change teh
reference (_readers is volatile).
+ // ArrayList is non-threadsafe. So add to a new copy and then change the
reference (_readers is volatile).
List<FixedByteSingleValueMultiColReader> readers = new
ArrayList<>(_readers);
readers.add(reader);
_readers = readers;
diff --git
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/nativefst/automaton/RegExp.java
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/nativefst/automaton/RegExp.java
index f04187ca4c2..178d609f7f4 100644
---
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/nativefst/automaton/RegExp.java
+++
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/nativefst/automaton/RegExp.java
@@ -82,7 +82,7 @@ public class RegExp {
* Constructs new <code>RegExp</code> from a string.
* Same as <code>RegExp(s, ALL)</code>.
* @param inputString regexp string
- * @exception IllegalArgumentException if an error occured while parsing the
regular expression
+ * @exception IllegalArgumentException if an error occurred while parsing
the regular expression
*/
public RegExp(String inputString)
throws IllegalArgumentException {
@@ -93,7 +93,7 @@ public class RegExp {
* Constructs new <code>RegExp</code> from a string.
* @param inputString regexp string
* @param syntaxFlags boolean 'or' of optional syntax constructs to be
enabled
- * @exception IllegalArgumentException if an error occured while parsing the
regular expression
+ * @exception IllegalArgumentException if an error occurred while parsing
the regular expression
*/
public RegExp(String inputString, int syntaxFlags)
throws IllegalArgumentException {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
