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 2020/11/26 07:42:41 UTC

[GitHub] [pulsar] holmes07 opened a new issue #8718: protobuf compile error

holmes07 opened a new issue #8718:
URL: https://github.com/apache/pulsar/issues/8718


   **Describe the bug**
   1. add one option fileds at proto file 
   2. rebuild it by exe :  /d/dev/protobuf-2.4.1/protoc --java_out=src/main/java src/main/proto/PulsarApi.proto
   
   but found error like this ,some method are removed( herer is recycle() )
   ![企业微信截图_16063763149492](https://user-images.githubusercontent.com/5210343/100321363-88021000-2ffd-11eb-941e-27510819b62c.png)
   
   so which version protobuf should i use ??? 
   Is anything wrong with 2.4.1 version any ? 
   


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



[GitHub] [pulsar] holmes07 commented on issue #8718: protobuf compile error

Posted by GitBox <gi...@apache.org>.
holmes07 commented on issue #8718:
URL: https://github.com/apache/pulsar/issues/8718#issuecomment-734133291


   
   It doesn't work even according to the document protobuf/readme.md . Can anyone provide a working solution? 
   


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



[GitHub] [pulsar] holmes07 commented on issue #8718: protobuf compile error

Posted by GitBox <gi...@apache.org>.
holmes07 commented on issue #8718:
URL: https://github.com/apache/pulsar/issues/8718#issuecomment-737017089


   > @holmes07
   > 
   > Please use this following command. It is documented in the README.
   > 
   > ```
   > cd ${PULSAR_HOME}/pulsar-common/
   > ./generate_protobuf_docker.sh
   > ```
   > 
   > @Jennifer88huang We might consider update the protobuf/readme.md file to make it clear.
   
   got it . i will try it later.  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.

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



[GitHub] [pulsar] sijie closed issue #8718: protobuf compile error

Posted by GitBox <gi...@apache.org>.
sijie closed issue #8718:
URL: https://github.com/apache/pulsar/issues/8718


   


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



[GitHub] [pulsar] sijie commented on issue #8718: protobuf compile error

Posted by GitBox <gi...@apache.org>.
sijie commented on issue #8718:
URL: https://github.com/apache/pulsar/issues/8718#issuecomment-737011310


   @holmes07 
   
   Please use this following command. It is documented in the README.
   
   ```
   cd ${PULSAR_HOME}/pulsar-common/
   ./generate_protobuf_docker.sh
   ```
   
   ---
   
   @Jennifer88huang We might consider update the protobuf/readme.md file to make it clear.


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