Fix serviceUrl for functions cli (#267)

* Use `distributedlog-core-shaded` in pulsar worker

* revert to db ledger storage

* Include netty-all

* Fix serviceUrl for functions cli

pulsar-functions/cli/src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java

@@ -445,7 +445,7 @@ public class CmdFunctions extends CmdBase {
                 throw new RuntimeException("Missing arguments");
             }
 
-            String serviceUrl = admin.getServiceUrl().toString();
+            String serviceUrl = ((PulsarFunctionsAdmin) admin).getClientConf().getConfigurationData().getServiceUrl();
             if (brokerServiceUrl != null) {
                 serviceUrl = brokerServiceUrl;
             }

pulsar-functions/cli/src/main/java/org/apache/pulsar/admin/cli/FunctionsTool.java

@@ -20,6 +20,7 @@ package org.apache.pulsar.admin.cli;
 
 import java.net.URL;
 import java.util.function.BiFunction;
+import org.apache.commons.lang3.StringUtils;
 import org.apache.pulsar.client.admin.PulsarAdmin;
 import org.apache.pulsar.client.admin.PulsarFunctionsAdmin;
 
@@ -36,6 +37,11 @@ public class FunctionsTool extends PulsarAdminTool {
     FunctionsTool(Properties properties) throws Exception {
         super(properties);
         commandMap.put("functions", CmdFunctions.class);
+
+        // in pulsar functions tool, webServiceUrl points to functions rest endpoint,
+        // which can not be used for constructing pulsar client. we have to use
+        // `serviceUrl` defined in the properties file.
+        config.setServiceUrl(properties.getProperty("serviceUrl"));
     }
 
     public static void main(String[] args) throws Exception {