This is an automated email from the ASF dual-hosted git repository.

technoboy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new fb9c4d034bf [fix][fn] Make pulsar-admin support update py/go with 
package url (#19897)
fb9c4d034bf is described below

commit fb9c4d034bfdcfdf256720ec2490c737cae53839
Author: jiangpengcheng <scjiangpengch...@gmail.com>
AuthorDate: Wed May 10 21:16:17 2023 +0800

    [fix][fn] Make pulsar-admin support update py/go with package url (#19897)
---
 .../src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java       | 4 ++++
 1 file changed, 4 insertions(+)

diff --git 
a/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java
 
b/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java
index 91ec7183ff1..9b30d59f167 100644
--- 
a/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java
+++ 
b/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java
@@ -1037,6 +1037,10 @@ public class CmdFunctions extends CmdBase {
             updateOptions.setUpdateAuthData(updateAuthData);
             if (Utils.isFunctionPackageUrlSupported(functionConfig.getJar())) {
                 getAdmin().functions().updateFunctionWithUrl(functionConfig, 
functionConfig.getJar(), updateOptions);
+            } else if 
(Utils.isFunctionPackageUrlSupported(functionConfig.getPy())) {
+                getAdmin().functions().updateFunctionWithUrl(functionConfig, 
functionConfig.getPy(), updateOptions);
+            } else if 
(Utils.isFunctionPackageUrlSupported(functionConfig.getGo())) {
+                getAdmin().functions().updateFunctionWithUrl(functionConfig, 
functionConfig.getGo(), updateOptions);
             } else {
                 getAdmin().functions().updateFunction(functionConfig, 
userCodeFile, updateOptions);
             }

Reply via email to