TisonKun commented on issue #9820: [FLINK-14290] Decouple plan translation from 
job execution/ClusterClient
URL: https://github.com/apache/flink/pull/9820#issuecomment-536957176
 
 
   Give it a quick review here are 2 thoughts
   
   1. where is `FlinkPlan` now?
   
   2. I think although the code conflicts with FLINK-14263 and FLINK-14130 they 
are not logic conflict. FLINK-14263 and FLINK-14130 are simple code refactors 
which might be checked in first and this one can rebase on that. Or we can 
defer the refactor until this one accepted. Either is OK to me.

----------------------------------------------------------------
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

Reply via email to