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 2025/10/23 00:33:53 UTC
(iceberg) 05/09: adjust test
This is an automated email from the ASF dual-hosted git repository.
singhpk234 pushed a commit to branch feature/serialize-bound-expression
in repository https://gitbox.apache.org/repos/asf/iceberg.git
commit 870d601ecc51e752b799a9cee0e4e80487e79c43
Author: Prashant Singh <pr...@snowflake.com>
AuthorDate: Tue Oct 14 12:04:03 2025 -0700
adjust test
---
.../main/java/org/apache/iceberg/expressions/ResolvedTransform.java | 5 +++--
.../java/org/apache/iceberg/expressions/TestResolvedReference.java | 3 ++-
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/api/src/main/java/org/apache/iceberg/expressions/ResolvedTransform.java b/api/src/main/java/org/apache/iceberg/expressions/ResolvedTransform.java
index b24cd3ece3..58d812833c 100644
--- a/api/src/main/java/org/apache/iceberg/expressions/ResolvedTransform.java
+++ b/api/src/main/java/org/apache/iceberg/expressions/ResolvedTransform.java
@@ -51,10 +51,11 @@ public class ResolvedTransform<S, T> implements UnboundTerm<T> {
try {
ValidationException.check(
transform.canTransform(boundRef.type()),
- "Cannot bind: %s cannot transform %s values from '%s'",
+ "Cannot bind: %s cannot transform %s values from '%s' with field id %s",
transform,
boundRef.type(),
- ref.name());
+ ref.name(),
+ ref.fieldId());
} catch (IllegalArgumentException e) {
throw new ValidationException(
"Cannot bind: %s cannot transform %s values from '%s'",
diff --git a/api/src/test/java/org/apache/iceberg/expressions/TestResolvedReference.java b/api/src/test/java/org/apache/iceberg/expressions/TestResolvedReference.java
index 9b03a4e8ea..9a9d84db1f 100644
--- a/api/src/test/java/org/apache/iceberg/expressions/TestResolvedReference.java
+++ b/api/src/test/java/org/apache/iceberg/expressions/TestResolvedReference.java
@@ -82,7 +82,8 @@ public class TestResolvedReference {
assertThatThrownBy(() -> ref.bind(SCHEMA.asStruct(), true))
.isInstanceOf(ValidationException.class)
- .hasMessageContaining("Cannot find field 'invalid' in struct");
+ .hasMessageContaining(
+ "Cannot find field with id '999' in struct: struct<34: a: optional int, 35: s: required string>, since we are resolving based on ID");
}
@Test