[GitHub] rabbah commented on issue #4134: the `processAcknowledgement` may process `CompletionMessage` first for blocking incovation even the message is sent ahead of `ResultMessage`

[GitBox <gi...@apache.org>]

rabbah commented on issue #4134: the `processAcknowledgement` may process `CompletionMessage` first for blocking incovation even the message is sent ahead of `ResultMessage`
URL: https://github.com/apache/incubator-openwhisk/issues/4134#issuecomment-441786004
 
 
   The DB poll is periodic and there is a race between getting the results from the poller vs active ack. Until the poll is cancelled (which is scheduled by the active ack), it remains active. Further, the future which continues with its callback could be executed in any order. Because of how close the log statements appear in time, I don't think the effect is necessary what you might be hinting at: that the result message is ignored for completing the blocking request --- which you can assert with a test otherwise.
   
   I saw you opened a PR so will take a look at that as well.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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