[GitHub] [openwhisk] bdoyle0182 commented on issue #4874: add activationId to events message

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

bdoyle0182 commented on issue #4874: add activationId to events message
URL: https://github.com/apache/openwhisk/pull/4874#issuecomment-612135057
 
 
   > Just to confirm the impact on backward compatability.
   > 
   > * Any existing code which is processing the user-events using older version of Message parsing code would work as is as it would just ignore activationId
   > * New code reading older events - Should not happen in general
   > 
   > Should we account for case where parser should be able to handle missing activationId i.e. mark it as `Option`
   
   Not sure if it needs to be an `Option` as any existing external service to openwhisk parsing the events topic just wouldn't look for activationId right? It would just ignore until it started looking for the field.

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