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 2021/08/31 22:05:41 UTC

[GitHub] [pulsar] nlu90 opened a new pull request #11868: [function] enable protobuf-native schema support for function

nlu90 opened a new pull request #11868:
URL: https://github.com/apache/pulsar/pull/11868


   Fixes #11721 
   
   ### Motivation
   
   Enable function proces topic with protobuf_native schema
   
   ### Modifications
   
   update `TopicSchema` 
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   
   Verified locally, the function is able to log the incoming message
   ```
   Received Person: name: "SN-EM-0"
   email: "0sn@streamnative.io"
   
   Received Person: name: "SN-EM-1"
   id: 1
   email: "1sn@streamnative.io"
   
   Received Person: name: "SN-EM-2"
   id: 2
   email: "2sn@streamnative.io"
   ```
   
   
   ### Documentation
   
   Check the box below and label this PR (if you have committer privilege).
   
   Need to update docs? 
   
   - [ ] doc-required 
     
     (If you need help on updating docs, create a doc issue)
     
   - [ ] no-need-doc 
     
     internal fix
     
   - [ ] doc 
     
     (If this PR contains doc changes)
   
   
   


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

[GitHub] [pulsar] nlu90 commented on pull request #11868: [function] enable protobuf-native schema support for function

Posted by GitBox <gi...@apache.org>.
nlu90 commented on pull request #11868:
URL: https://github.com/apache/pulsar/pull/11868#issuecomment-910999091


   /pulsarbot run-failure-checks


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

[GitHub] [pulsar] sijie merged pull request #11868: [function] enable protobuf-native schema support for function

Posted by GitBox <gi...@apache.org>.
sijie merged pull request #11868:
URL: https://github.com/apache/pulsar/pull/11868


   


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

[GitHub] [pulsar] nlu90 commented on pull request #11868: [function] enable protobuf-native schema support for function

Posted by GitBox <gi...@apache.org>.
nlu90 commented on pull request #11868:
URL: https://github.com/apache/pulsar/pull/11868#issuecomment-910863558


   > Could we add a test for the schema `PROTOBUF_NATIVE`?
   
   added a test class for `TopicSchema`. PTAL


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