This is an automated email from the ASF dual-hosted git repository.
wenchen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 74bab76cef3b [SPARK-55929][SQL] Add missing toString() to
TableChange.UpdateColumnDefaultValue
74bab76cef3b is described below
commit 74bab76cef3beb69d253cc7d736e83157a63ab89
Author: anshul_baliga7 <[email protected]>
AuthorDate: Tue Mar 10 22:13:24 2026 +0800
[SPARK-55929][SQL] Add missing toString() to
TableChange.UpdateColumnDefaultValue
### What changes were proposed in this pull request?
Add missing `toString()` to `TableChange.UpdateColumnDefaultValue`, which
was overlooked in SPARK-55828.
### Why are the changes needed?
SPARK-55828 added `toString()` to all `ColumnChange` implementations to
improve error messages in schema evolution. `UpdateColumnDefaultValue`
implements `ColumnChange` but was missed.
### Does this PR introduce _any_ user-facing change?
Yes. When `UpdateColumnDefaultValue` appears in a schema evolution error,
it now shows a readable sql string instead of a Java object reference.
### How was this patch tested?
Follows the same pattern as SPARK-55828.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #54721 from anshulbaliga7/add_toString_UpdateColumnDefaultValue.
Authored-by: anshul_baliga7 <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
---
.../java/org/apache/spark/sql/connector/catalog/TableChange.java | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git
a/sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/TableChange.java
b/sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/TableChange.java
index e64fa5e64903..1dec6c892ea1 100644
---
a/sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/TableChange.java
+++
b/sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/TableChange.java
@@ -810,6 +810,14 @@ public interface TableChange {
result = 31 * result + Objects.hashCode(newCurrentDefault);
return result;
}
+
+ @Override
+ public String toString() {
+ return newCurrentDefault == null
+ ? "ALTER COLUMN " + TableChange.fieldPath(fieldNames) + " DROP
DEFAULT"
+ : "ALTER COLUMN " + TableChange.fieldPath(fieldNames) + " SET
DEFAULT " +
+ newCurrentDefault.getSql();
+ }
}
/**
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
