Re: [DISCUSS] Drop "canEqual" from TypeInformation, TypeSerializer, etc.

2018-07-11 Thread Aljoscha Krettek
I created https://issues.apache.org/jira/browse/FLINK-9798 > On 18. May 2018, at 16:10, Till Rohrmann wrote: > > +1 > > If we don't have a hierarchy of serializers then this method has no right > to exist. > > Cheers, > Till > > On Wed, May 16, 2018 at 11:06 AM, Timo Walther wrote: > >> +1

Re: [DISCUSS] Drop "canEqual" from TypeInformation, TypeSerializer, etc.

2018-05-18 Thread Till Rohrmann
+1 If we don't have a hierarchy of serializers then this method has no right to exist. Cheers, Till On Wed, May 16, 2018 at 11:06 AM, Timo Walther wrote: > +1 > > TypeInformation has too many methods that need to be implemented but > provide little benefit for Flink. > >

Re: [DISCUSS] Drop "canEqual" from TypeInformation, TypeSerializer, etc.

2018-05-16 Thread Timo Walther
+1 TypeInformation has too many methods that need to be implemented but provide little benefit for Flink. Am 16.05.18 um 10:55 schrieb Ted Yu: +1 from me as well. I checked a few serializer classes. The `equals` method on serializers contains the logic of `canEqual` method whose existence

Re: [DISCUSS] Drop "canEqual" from TypeInformation, TypeSerializer, etc.

2018-05-16 Thread Ted Yu
+1 from me as well. I checked a few serializer classes. The `equals` method on serializers contains the logic of `canEqual` method whose existence seems redundant. On Wed, May 16, 2018 at 1:49 AM, Tzu-Li (Gordon) Tai wrote: > +1. > > Looking at the implementations of the

Re: [DISCUSS] Drop "canEqual" from TypeInformation, TypeSerializer, etc.

2018-05-16 Thread Tzu-Li (Gordon) Tai
+1. Looking at the implementations of the `canEqual` method in several serializers, it seems like all that is done is a check whether the object is of the same serializer class. We’ll have to be careful and double check all `equals` method on serializers that may have relied on the `canEqual`

[DISCUSS] Drop "canEqual" from TypeInformation, TypeSerializer, etc.

2018-05-16 Thread Stephan Ewen
Hi all! As part of an attempt to simplify some code in the TypeInfo and TypeSerializer area, I would like to drop the "canEqual" methods for the following reason: "canEqual()" is necessary to make proper equality checks across hierarchies of types. This is for example useful in a collection API,