Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error report of udf server connection can be more clear and user-friendly #13163

Closed
keven-huang opened this issue Oct 31, 2023 · 2 comments
Closed
Assignees
Milestone

Comments

@keven-huang
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Description

When I use sql CREATE FUNCTION gcd(int, int) RETURNS int AS gcd USING LINK '10.10.4.17:8815'; to call the udf-server deployed in Kubernetes, I encountered an error Internal error: failed to connect to UDF service: transport error
Later, I found that I needed to add 'http://' to the link to change it from '10.10.4.17:8815' to 'http://10.10.4.17:8815'.
After discussing with @arkbriar , we both agree that this error message is not very user-friendly and could be improved.

related link

https://docs.risingwave.com/docs/current/udf-java/#5-declare-your-functions-in-risingwave

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

@BugenZhao
Copy link
Member

This is not a trivial issue, and the root cause is that we lose the source error when formatting.

#11443 (comment)

@BugenZhao BugenZhao assigned BugenZhao and unassigned wangrunji0408 Nov 6, 2023
@BugenZhao
Copy link
Member

After #13248 and #13264, we'll get a much more detailed error reporting:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants