You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2019/09/06 15:20:31 UTC

[GitHub] [flink] 1u0 commented on issue #9634: [hotfix][runtime] Streaming runtime 1.9 improvements for debuggability

1u0 commented on issue #9634: [hotfix][runtime] Streaming runtime 1.9 improvements for debuggability
URL: https://github.com/apache/flink/pull/9634#issuecomment-528897588
 
 
   The change with the first commit is already present in the `master` branch.
   If the second commit (with renaming of `performDefaultAction()`) is also ok to introduce in the `release-1.9` branch, then I recommend to merge this PR as is. But not backport it into `master` branch (!).
   
   I have opened another PR #9635 to port the second commit into `master` branch. Otherwise, directly rebasing the second commit against the `master` branch should have merge conflicts (imo), as there was some refactoring that removed some `performDefaultAction()` overrides.

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


With regards,
Apache Git Services