[GitHub] [pulsar] jerrypeng edited a comment on issue #4093: [issue#4042] improve java functions API

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

jerrypeng edited a comment on issue #4093: [issue#4042] improve java functions API
URL: https://github.com/apache/pulsar/pull/4093#issuecomment-485080376
 
 
   > The only function code involve in current context.publish is “this.statsManager.incrSysExceptions(e);”.
   
   Yes but we need this.
   
   @sijie what do you also propose on how to set these message configs for outputs returned by the function?  
   
   That is why I proposed some like the following because then it will be uniform:
   
   ```
   MessageConf messageConf = MessageConf.Builder().key("key").properties(myProperties).build()
   
   context.publish("output", "my-topic", messageConf)
   
   # or for output message
   
   context.setOutputMessageConf(messageConf)
   ```
   
   

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