godfreyhe edited a comment on issue #9433: [FLINK-13708] [table-planner-blink] 
transformations should be cleared after execution in blink planner
URL: https://github.com/apache/flink/pull/9433#issuecomment-521680333
 
 
   > @godfreyhe I see the problem. TBH I would prefer the approach you 
suggested more than how it currently is implemented. It supports the old 
behavior.
   
   I find another problem today: if `explain` twice, the second result contains 
the first result, or if `execute` after `explain`, the `StreamGraph` to execute 
also contains transformations generated in `explain`. This bug exists in both 
planners.
   
   the latest fix is: generate `StreamGraph` not through 
`StreamExecutionEnvironment` but `StreamGraphGenerator`. This approach could 
avoid `StreamExecutionEnvironment` buffer the transformations added in 
`explain` method or in `execute` method, and could generate clean 
`StreamGraph`. This will be fixed in blink planner now.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to