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 <[email protected]> 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
