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 2022/11/30 09:26:18 UTC

[GitHub] [flink-kubernetes-operator] gaborgsomogyi commented on pull request #438: [FLINK-29974] Not allowing the cancelling the which are already in the completed state.

gaborgsomogyi commented on PR #438:
URL: https://github.com/apache/flink-kubernetes-operator/pull/438#issuecomment-1331864650

   From directional perspective it looks good but I need to go through all the details.
   In the meantime I would like to ask you 2 things:
   * Please rebase the code to the latest master to have important changes like [this](https://github.com/apache/flink-kubernetes-operator/blob/80e5f3f4add50355de6964c1466ba558d46bc79d/e2e-tests/test_multi_sessionjob.sh#L41)
   * Could you please squash all the commits into a single one to ease life (force pushing this branch is good in general)
   


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

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org