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

[#4845] fix(core): Fix bugs when updating metalake or catalog with nullable comment. #4855

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Sep 4, 2024

What changes were proposed in this pull request?

Change the update SQL in the mapper to handle nullable comment.

Why are the changes needed?

It's a bug to fix.

Fix: #4845

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

Added UTs and ITs

…llable comment. (#4846)

### What changes were proposed in this pull request?

Change the update SQL in the mapper to handle nullable comment. 

### Why are the changes needed?

It's a bug to fix.

Fix: #4845 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Added UTs and ITs
@jerryshao
Copy link
Contributor

One CI is hang due to unexpected reason, merging it anyway.

@jerryshao jerryshao merged commit cab9eff into branch-0.6 Sep 4, 2024
38 of 39 checks passed
@jerryshao jerryshao deleted the cherry-pick-branch-0.6-8ae10cc1cd664f9836c9ce268e3278f8c3ae7d14 branch September 4, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants