c4emmmm commented on issue #8402: [FLINK-12473][ml] Add the interface of ML pipeline and ML lib URL: https://github.com/apache/flink/pull/8402#issuecomment-492198604 Thank @Xpray , @walterddr , @bowenli86 , @ex00 for your advice. I have pushed a new commit modifying Pipeline. The changes are: Make stages final, and getStages() returns an immutable copy of the stages. New pipeline with a pipelineStages list and restoring from json also use appendStage(). (These make the stages only editable via the appendStage() method.) Validate and record index of last estimator in appendStage(), then all stages will be guaranteed valid and there'll be no need to find the index of last estimator many times.
---------------------------------------------------------------- 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] With regards, Apache Git Services
