sijie commented on issue #3854: [issue #3767] support go function for pulsar URL: https://github.com/apache/pulsar/pull/3854#issuecomment-479993556 @srkukarni : I don't think your comments around error handling for EO are needed here. technically the whole PR doesn't handle EO. EO requires special setup on producers. EO itself is a huge task. I would prefer leaving EO discussion out of this PR so that we can have a MVP for go instance first. Then we can iterate from there. With that being said, I think we should do followings: 1) fail the instance (panic) in newGoInstance if ProcessingGuarantee (PG) is EO. 2) remove the error handle for EO. only handle AMO and ALO.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
