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 2020/10/14 03:09:48 UTC

[GitHub] [flink-statefun] tzulitai opened a new pull request #163: [FLINK-19620] Merge ExactlyOnceE2E and RemoteModuleE2E

tzulitai opened a new pull request #163:
URL: https://github.com/apache/flink-statefun/pull/163


   The original `ExactlyOnceE2E` verified end-to-end exactly-once with TM failures, while the `RemoteModuleE2E` verified a deployment with remote functions.
   
   As mentioned in #159 by @sjwiesman, we seem to be lacking an exactly-once verifying E2E with remote functions.
   
   It is actually worthwhile to merge these 2 E2Es into one, since 1) remote functions are built on top of embedded functions, so merging them does not hurt us on test coverage, and 2) E2E tests take a significant amount of test time, so less is better.
   
   ### Change log
   
   - bbe5d84: Extends the original `RemoteModuleE2E` to have TM failures, `read_committed` settings for Kafka consumers that verify outputs, and exactly-once delivering Kafka egresses.
   - 1c022ad: Removes the `ExactlyOnceE2E`
   - 8ca8993: Renames `RemoteModuleE2E` to `ExactlyOnceWithRemoteFnE2E` to reflect the extra coverage in the test.


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



[GitHub] [flink-statefun] tzulitai commented on pull request #163: [FLINK-19620] Merge ExactlyOnceE2E and RemoteModuleE2E

Posted by GitBox <gi...@apache.org>.
tzulitai commented on pull request #163:
URL: https://github.com/apache/flink-statefun/pull/163#issuecomment-708128215


   cc @sjwiesman @igalshilman 


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



[GitHub] [flink-statefun] tzulitai closed pull request #163: [FLINK-19620] Merge ExactlyOnceE2E and RemoteModuleE2E

Posted by GitBox <gi...@apache.org>.
tzulitai closed pull request #163:
URL: https://github.com/apache/flink-statefun/pull/163


   


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