You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/06/14 13:35:02 UTC

[GitHub] [pulsar] cbornet commented on pull request #16041: [improve][function] Support Record as Function output type

cbornet commented on PR #16041:
URL: https://github.com/apache/pulsar/pull/16041#issuecomment-1155196275

   > Is this change ABI compatible so that existing user created Pulsar Functions implementations can be run without recompiling when upgrading to a version that includes this change?
   
   @lhotari I think it is since we only add a method to the `Context` API. 
   `SinkRecord` is sometimes used outside of the package (and probably shouldn't) in tests but I think extracting some methods to a super class is OK for binary compat. Or is it not ?
   Do you see other things that could break ?


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org