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/07/20 16:46:47 UTC

(iceberg) branch main updated: API: Shorten dropped source type comment in PartitionSpec (follow-up to #17262) (#17303)

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 2f17d9bafd API: Shorten dropped source type comment in PartitionSpec (follow-up to #17262) (#17303)
2f17d9bafd is described below

commit 2f17d9bafdb643819530ec21371fd1380438303e
Author: Anoop Johnson <an...@apache.org>
AuthorDate: Mon Jul 20 09:46:40 2026 -0700

    API: Shorten dropped source type comment in PartitionSpec (follow-up to #17262) (#17303)
---
 api/src/main/java/org/apache/iceberg/PartitionSpec.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/api/src/main/java/org/apache/iceberg/PartitionSpec.java b/api/src/main/java/org/apache/iceberg/PartitionSpec.java
index c2cf559145..f0b1051ff5 100644
--- a/api/src/main/java/org/apache/iceberg/PartitionSpec.java
+++ b/api/src/main/java/org/apache/iceberg/PartitionSpec.java
@@ -191,9 +191,8 @@ public class PartitionSpec implements Serializable {
   private Type resultType(PartitionField field) {
     Type sourceType = schema.findType(field.sourceId());
     if (sourceType == null) {
-      // when the source field has been dropped, substitute unknown and let the transform derive
-      // its result type; transforms that return the source type (identity, truncate, void) yield
-      // unknown, while transforms with a fixed result type still resolve
+      // When the source field has been dropped, the source type has been lost
+      // Transforms with a fixed result type still work, so use unknown for source
       sourceType = Types.UnknownType.get();
     }