twalthr commented on a change in pull request #13967:
URL: https://github.com/apache/flink/pull/13967#discussion_r518821443
##########
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:
We fixed MapData in FLINK-17668. I think most internal data structures
have a proper hashCode/equals now. But yes, we can also be more specific here.
----------------------------------------------------------------
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]
