amaliujia commented on a change in pull request #13920:
URL: https://github.com/apache/beam/pull/13920#discussion_r585116358
##########
File path: sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/Schema.java
##########
@@ -949,6 +950,40 @@ public int hashCode() {
getMetadata()
});
}
+
+ @Override
+ public String toString() {
Review comment:
I see. In this case, do you think if it is worth also having simple
tests for the new toString() code added (just for MAP,ROW,ARRAY)?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
