[GitHub] merlimat commented on a change in pull request #706: Add missing lib instrunction for pulsar-ml-admin tool

[gi...@git.apache.org]

merlimat commented on a change in pull request #706: Add missing lib instrunction for pulsar-ml-admin tool
URL: https://github.com/apache/incubator-pulsar/pull/706#discussion_r134062612
 
 

 ##########
 File path: bin/pulsar-managed-ledger-admin
 ##########
 @@ -20,11 +20,19 @@
 
 import argparse
 import traceback
-from google.protobuf.text_format import Merge
-from google.protobuf.text_format import MessageToString
 import sys
-    
-from proto import MLDataFormats_pb2
+try:
+    from google.protobuf.text_format import Merge
+    from google.protobuf.text_format import MessageToString
+except Exception as missingLib:
+    sys.exit("You need python protobuf library. See: https://github.com/google/protobuf/tree/master/python")
 
 Review comment:
   You should be able to get that from PyPI as well, right? 
   `pip install protobuf`
 
----------------------------------------------------------------
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