+1 (binding) Best, Aljoscha
> On 15. Oct 2019, at 04:01, Zili Chen <wander4...@gmail.com> wrote: > > Hi all, > > +1 from my side. > > Given the current state of this voting thread, FLIP-74 is accepted > with 3 binding vote and 2 non-binding vote. Thanks for your > participation! > > I will update the wiki to reflect that the result of the vote. > > Best, > tison. > > > Zili Chen <wander4...@gmail.com> 于2019年10月11日周五 下午8:48写道: > >> Well. Then I'd remove the requirement to change cancelWithSavepoint >> but remain why we exclude it from JobClient. >> >> We might still change signature to completable future for a consistent >> async view of ClusterClient but it is quite an implement detail and we >> don't stick to it on FLIP level. >> >> Best, >> tison. >> >> >> Kostas Kloudas <kklou...@gmail.com> 于2019年10月11日周五 下午7:36写道: >> >>> Hi Tison, >>> >>> Thanks for integrating the comments! >>> >>> +1 for accepting the FLIP from my side. >>> What I meant is that in the Proposed Changes section, the FLIP still >>> has that the cancelWithSavepoin(jobId, savepointDir) of the >>> clusterClient should change to return a CompletableFuture. I believe >>> that this change is redundant as we will not need it for the >>> JobClient. I should have been more clear on what I meant before. >>> >>> Cheers, >>> Kostas >>> >>> On Fri, Oct 11, 2019 at 11:51 AM Zili Chen <wander4...@gmail.com> wrote: >>>> >>>> Hi Kostas, >>>> >>>> Thanks for your reply. >>>> >>>> (1) cancelWithSavepoint() has already been excluded from the FLIP. But >>>> to emphasize that we make the decision to exclude it I add it to reject >>>> alternatives. >>>> >>>> (2) Updated FLIP to reflect the consensus :-) >>>> >>>> Best, >>>> tison. >>>> >>>> >>>> Kostas Kloudas <kklou...@gmail.com> 于2019年10月11日周五 下午5:12写道: >>>> >>>>> Hi all, >>>>> >>>>> I only have two minor comments before voting and they have to do with >>>>> the following: >>>>> >>>>> 1) In the discussion, we agreed to remove the cancelWithSavepoint() >>>>> from the JobClient as this is deprecated in the rest API. This is not >>>>> in the FLIP. >>>>> 2) The section "ClusterDescriptor or Executor(FLIP-73)(integration)" >>>>> does not reflect our discussion where we said that for now only the >>>>> Executor#execute() will give you the JobClient and there will be a >>>>> separate discussion about alternative ways of exposing the JobClient. >>>>> >>>>> I think that these points should be updated in order for the FLIP to >>>>> reflect the discussion in the ML thread. >>>>> >>>>> Cheers, >>>>> Kostas >>>>> >>>>> On Fri, Oct 11, 2019 at 10:58 AM Biao Liu <mmyy1...@gmail.com> wrote: >>>>>> >>>>>> +1 (non-binding), glad to have this improvement! >>>>>> >>>>>> Thanks, >>>>>> Biao /'bɪ.aʊ/ >>>>>> >>>>>> >>>>>> >>>>>> On Fri, 11 Oct 2019 at 14:44, Jeff Zhang <zjf...@gmail.com> wrote: >>>>>> >>>>>>> +1, overall design make sense to me >>>>>>> >>>>>>> SHI Xiaogang <shixiaoga...@gmail.com> 于2019年10月11日周五 上午11:15写道: >>>>>>> >>>>>>>> +1. The interface looks fine to me. >>>>>>>> >>>>>>>> Regards, >>>>>>>> Xiaogang >>>>>>>> >>>>>>>> Zili Chen <wander4...@gmail.com> 于2019年10月9日周三 下午2:36写道: >>>>>>>> >>>>>>>>> Given the ongoing FlinkForward Berlin event, I'm going to >>> extend >>>>>>>>> this vote thread with a bit of period, said until Oct. >>>>> 11th(Friday). >>>>>>>>> >>>>>>>>> Best, >>>>>>>>> tison. >>>>>>>>> >>>>>>>>> >>>>>>>>> Zili Chen <wander4...@gmail.com> 于2019年10月7日周一 下午4:15写道: >>>>>>>>> >>>>>>>>>> Hi all, >>>>>>>>>> >>>>>>>>>> I would like to start the vote for FLIP-74[1], which is >>>>> discussed and >>>>>>>>>> reached a consensus in the discussion thread[2]. >>>>>>>>>> >>>>>>>>>> The vote will be open util Oct. 9th(72h starting on >>> Oct.7th), >>>>> unless >>>>>>>>>> there is an objection or not enough votes. >>>>>>>>>> >>>>>>>>>> Best, >>>>>>>>>> tison. >>>>>>>>>> >>>>>>>>>> [1] >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>> >>> https://cwiki.apache.org/confluence/display/FLINK/FLIP-74%3A+Flink+JobClient+API >>>>>>>>>> [2] >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>> >>> https://lists.apache.org/x/thread.html/b2e22a45aeb94a8d06b50c4de078f7b23d9ff08b8226918a1a903768@%3Cdev.flink.apache.org%3E >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Best Regards >>>>>>> >>>>>>> Jeff Zhang >>>>>>> >>>>> >>> >>