-
Notifications
You must be signed in to change notification settings - Fork 637
fix: udf error ui #18118
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
fix: udf error ui #18118
Conversation
Signed-off-by: Richard Chien <[email protected]>
45693e1
to
b83880c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why CI didn’t prevent the regression? Is this a flaky test..?
I believe this can be caused by CI infra's network configuration change. |
Signed-off-by: Richard Chien <[email protected]>
Signed-off-by: Richard Chien <[email protected]>
Another CI test produced the old message: https://buildkite.com/risingwavelabs/pull-request/builds/56040#01916e22-fdda-4a17-aea8-bbd6e32453b0 I guess this is related to the network setup of the spot instance allocated for the CI pipeline. Is it possible to support only to check the first several lines of the error message in sqllogictest? |
You mean using the regex match syntax? |
Signed-off-by: Richard Chien <[email protected]>
I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Checklist
./risedev check
(or alias,./risedev c
)Documentation
Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.