You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/09/08 15:04:02 UTC

[GitHub] [pulsar] codelipenghui opened a new issue #8009: Number format exception when create function with multiple service url address

codelipenghui opened a new issue #8009:
URL: https://github.com/apache/pulsar/issues/8009


   **Describe the bug**
   Number format exception when create function with multiple service url address
   
   **To Reproduce**
   Steps to reproduce the behavior:
   1. Use pulsar-admin to createFunction and specify multiple service URL for the function config.
   2. See error
   
   **Expected behavior**
   Allow using multiple service URL when creating function
   
   **Additional context**
   2.6.1


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [pulsar] jianyun8023 commented on issue #8009: Number format exception when create function with multiple service url address

Posted by GitBox <gi...@apache.org>.
jianyun8023 commented on issue #8009:
URL: https://github.com/apache/pulsar/issues/8009#issuecomment-697635653


   The root of this problem is that PulsarAdmin does not support multiple multiple service url address.
   I think the solution to this problem is to **add validation of the `serviceHttpUrl` parameter**.
   @sijie @codelipenghui @zymap Everyone, do you think how to solve this problem is better?
   
   for example:
   ```
   final PulsarAdmin admin = PulsarAdmin.builder()
       .serviceHttpUrl("http://localhost:8080,http://localhost:8081")
       .build();
   FunctionConfig functionConfig = buildFunctionConfig();
   admin.functions().createFunction(functionConfig, "pulsar-functions-api-examples.jar");
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [pulsar] jianyun8023 commented on issue #8009: Number format exception when create function with multiple service url address

Posted by GitBox <gi...@apache.org>.
jianyun8023 commented on issue #8009:
URL: https://github.com/apache/pulsar/issues/8009#issuecomment-718954892


   @codelipenghui pr #8143 has solved the NumberException problem. The current question is whether to close or newly open Function to support the multiple service url address.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [pulsar] jianyun8023 commented on issue #8009: Number format exception when create function with multiple service url address

Posted by GitBox <gi...@apache.org>.
jianyun8023 commented on issue #8009:
URL: https://github.com/apache/pulsar/issues/8009#issuecomment-697597978


   > Hi @codelipenghui, could you please provide the create function command and the detail of the error message?
   
   I have reproduced the problem and I am solving it.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [pulsar] zymap commented on issue #8009: Number format exception when create function with multiple service url address

Posted by GitBox <gi...@apache.org>.
zymap commented on issue #8009:
URL: https://github.com/apache/pulsar/issues/8009#issuecomment-697271020


   Hi @codelipenghui, could you please provide the create function command and the detail of the error message? 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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