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

Posted to commits@pulsar.apache.org by gi...@git.apache.org on 2017/08/18 21:42:45 UTC

[GitHub] rdhabalia opened a new pull request #706: Add missing lib instrunction for pulsar-ml-admin tool

rdhabalia opened a new pull request #706: Add missing lib instrunction for pulsar-ml-admin tool
URL: https://github.com/apache/incubator-pulsar/pull/706
 
 
   ### Motivation
   
   Right now, `pulsar-managed-ledger-admin` tool can fail if `protoc-python` lib is not installed or generated `MLDataFormats.proto` is not compatible with current protoc library. And it will be hard for user to figure out the solution.
   
   ### Modifications
   
   Add instruction if `protoc` library is missing or `MLDataFormats.proto` incompatible with existing `protoc` library.
   
   ### Result
   
   `pulsar-managed-ledger-admin` tool will show the instruction on protocbuf import error.
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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