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/10 03:47:21 UTC

[GitHub] [pulsar] congbobo184 opened a new pull request #11615: Fix schema-understand doc.

congbobo184 opened a new pull request #11615:
URL: https://github.com/apache/pulsar/pull/11615


   now complex type example are not right, so fix it.
   
   


-- 
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] Anonymitaet commented on pull request #11615: Fix schema-understand doc.

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


   Thanks for your contribution. Does this affect only master or other versioned docs? 
   If latter, could you please help update all affected versions? Thanks


-- 
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] congbobo184 commented on pull request #11615: Fix schema-understand doc.

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


   @BewareMyPower hi, I think this example is used to explain struct schema. Both avro and protobuf are good, what do you think?


-- 
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] Anonymitaet merged pull request #11615: Fix schema-understand doc.

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


   


-- 
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] BewareMyPower commented on pull request #11615: Fix schema-understand doc.

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


   OK, I also just found the original doc PR is a little confused because it contains both the schema definition part and protobuf native schema part.


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