You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by si...@apache.org on 2026/08/30 21:21:44 UTC

(iceberg) branch main updated: API, Core: Fix typos in test comments (#17889)

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

singhpk234 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/main by this push:
     new 04c3921822 API, Core: Fix typos in test comments (#17889)
04c3921822 is described below

commit 04c39218226bbde5e07b1d169f2bf0eea35d11c6
Author: damansingh1313 <da...@gmail.com>
AuthorDate: Mon Aug 31 02:51:39 2026 +0530

    API, Core: Fix typos in test comments (#17889)
    
    Fix a duplicated "the", "overriden" -> "overridden", and "doesnt" -> "doesn't" in
    test comments. Comment-only changes, no behavior change.
---
 api/src/test/java/org/apache/iceberg/transforms/TestBucketing.java     | 2 +-
 core/src/test/java/org/apache/iceberg/TestSplitPlanning.java           | 2 +-
 .../java/org/apache/iceberg/catalog/TestTableIdentifierParser.java     | 3 +--
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/api/src/test/java/org/apache/iceberg/transforms/TestBucketing.java b/api/src/test/java/org/apache/iceberg/transforms/TestBucketing.java
index 81f4fa6098..d697d1548e 100644
--- a/api/src/test/java/org/apache/iceberg/transforms/TestBucketing.java
+++ b/api/src/test/java/org/apache/iceberg/transforms/TestBucketing.java
@@ -135,7 +135,7 @@ public class TestBucketing {
     assertThat(BucketUtil.hash(bytes))
         .as("Spec example: hash([00 01 02 03]) = -188683207")
         .isEqualTo(-188683207);
-    // another assertion for confirming that hashing a ByteBuffer again doesnt modify its value.
+    // another assertion for confirming that hashing a ByteBuffer again doesn't modify its value.
     assertThat(BucketUtil.hash(bytes))
         .as("Spec example: hash([00 01 02 03]) = -188683207")
         .isEqualTo(-188683207);
diff --git a/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java b/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java
index 82b3c4084d..89198687e5 100644
--- a/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java
+++ b/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java
@@ -141,7 +141,7 @@ public class TestSplitPlanning extends TestBase {
     List<DataFile> file128Mb = newFiles(1, 128 * 1024 * 1024);
     Iterable<DataFile> files = Iterables.concat(files120Mb, file128Mb);
     appendFiles(files);
-    // we expect 2 bins from non-overriden table properties
+    // we expect 2 bins from non-overridden table properties
     TableScan scan = table.newScan().option(TableProperties.SPLIT_LOOKBACK, "1");
     CloseableIterable<CombinedScanTask> tasks = scan.planTasks();
     assertThat(tasks).hasSize(2);
diff --git a/core/src/test/java/org/apache/iceberg/catalog/TestTableIdentifierParser.java b/core/src/test/java/org/apache/iceberg/catalog/TestTableIdentifierParser.java
index 739a87c241..9b44acf75b 100644
--- a/core/src/test/java/org/apache/iceberg/catalog/TestTableIdentifierParser.java
+++ b/core/src/test/java/org/apache/iceberg/catalog/TestTableIdentifierParser.java
@@ -56,8 +56,7 @@ public class TestTableIdentifierParser {
 
     String identifierMissingNamespace = "{\"name\":\"paid\"}";
     assertThat(TableIdentifierParser.fromJson(identifierMissingNamespace))
-        .as(
-            "Should implicitly convert a missing namespace into the the empty namespace when parsing")
+        .as("Should implicitly convert a missing namespace into the empty namespace when parsing")
         .isEqualTo(identifierWithEmptyNamespace);
   }