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/19 11:33:53 UTC
[GitHub] [flink-statefun] fransking opened a new pull request #165: [FLINK-19515][python] InvocationContext use after dispose fix
fransking opened a new pull request #165:
URL: https://github.com/apache/flink-statefun/pull/165
Moved creation of the IC out of __init__ and into __call__ so it is re-created per call.
----------------------------------------------------------------
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 #165: [FLINK-19515][python] InvocationContext use after dispose fix
Posted by GitBox <gi...@apache.org>.
tzulitai commented on pull request #165:
URL: https://github.com/apache/flink-statefun/pull/165#issuecomment-712600884
Merged:
2.2.1: e10cdd3218b56f75a40ae107994ca7463c55b5a9
2.3.0: b3a88dab20c56bc7cb1c1e2e2fe3237eda46144a
----------------------------------------------------------------
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] fransking closed pull request #165: [FLINK-19515][python] InvocationContext use after dispose fix
Posted by GitBox <gi...@apache.org>.
fransking closed pull request #165:
URL: https://github.com/apache/flink-statefun/pull/165
----------------------------------------------------------------
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] fransking commented on pull request #165: [FLINK-19515][python] InvocationContext use after dispose fix
Posted by GitBox <gi...@apache.org>.
fransking commented on pull request #165:
URL: https://github.com/apache/flink-statefun/pull/165#issuecomment-712096689
Tested with the orignal async examples and AsyncRequestReplyHandler now works as expected - i.e. no need to create an AsyncRequestReplyHandler per call.
----------------------------------------------------------------
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 #165: [FLINK-19515][python] InvocationContext use after dispose fix
Posted by GitBox <gi...@apache.org>.
tzulitai commented on pull request #165:
URL: https://github.com/apache/flink-statefun/pull/165#issuecomment-712598630
LGTM, thanks for fixing @fransking.
Will merge this 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:
users@infra.apache.org
[GitHub] [flink-statefun] tzulitai closed pull request #165: [FLINK-19515][python] InvocationContext use after dispose fix
Posted by GitBox <gi...@apache.org>.
tzulitai closed pull request #165:
URL: https://github.com/apache/flink-statefun/pull/165
----------------------------------------------------------------
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