jai1 opened a new pull request #1492: Removed Pulsar Functions from AdminTool
URL: https://github.com/apache/incubator-pulsar/pull/1492
 
 
   Currently in master:
   ```
   ./bin/pulsar-admin 
   class java.lang.reflect.InvocationTargetException: null
   ```
   Reason is that in `CmdFunctions.java:639` we expect the admin to be an 
instance of `PulsarAdminWithFunctions` while `PulsarAdminBuilderImpl.build()` 
builds an instance `PulsarAdmin`
   
   I tried building `PulsarAdminWithFunctions` in `PulsarAdminBuilderImpl` 
instead of `PulsarAdmin` but ran into lots of errors so I just commented out 
adding pulsar functions to `PulsarAdminTool`, in order to get master working 
since I need to develop on Admin Tools.
   
   I believe the problematic PR is 
https://github.com/apache/incubator-pulsar/commit/6230ab4542c733f3390bd7ae7d4d6b0d5a661fb1
 but can't say for sure since we use SNAPSHOT bookkeeper versions now.
   

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to