Re: [DISCUSS] Adding a dispose() method in the RichFunction.

2016-11-10 Thread Kostas Kloudas
I suppose that given that in the DataSet API there is no semantic distinction between the two, we just have to make sure that whenever close() is called, the new dispose() is called as well. Best, Kostas > On Nov 10, 2016, at 11:47 AM, Fabian Hueske wrote: > > RichFunctions are used in the

Re: [DISCUSS] Adding a dispose() method in the RichFunction.

2016-11-10 Thread Fabian Hueske
RichFunctions are used in the DataStream and DataSet APIs. How would that change affect the DataSet API? Best, Fabian 2016-11-10 11:37 GMT+01:00 Kostas Kloudas : > Hello, > > I would like to propose the addition of a dispose() method, in addition to > the > already existing close(), in the Rich

[DISCUSS] Adding a dispose() method in the RichFunction.

2016-11-10 Thread Kostas Kloudas
Hello, I would like to propose the addition of a dispose() method, in addition to the already existing close(), in the RichFunction interface. This will align the lifecycle of a RichFunction, with that of an Operator. After this, the code paths followed when finishing successfully and when ca