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);
}