wuchong commented on a change in pull request #13967:
URL: https://github.com/apache/flink/pull/13967#discussion_r518814704



##########
File path: flink-core/src/main/java/org/apache/flink/types/Row.java
##########
@@ -45,6 +45,9 @@
  *
  * <p>A row instance is in principle {@link Serializable}. However, it may 
contain non-serializable fields
  * in which case serialization will fail if the row is not serialized with 
Flink's serialization stack.
+ *
+ * <p>The {@link #equals(Object)} and {@link #hashCode()} methods of this 
class support all conversion

Review comment:
       I think you mean "all **external** conversion classes"? Because it 
doesn't support the internal conversion classes, e.g. `MapData`. 




----------------------------------------------------------------
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]


Reply via email to