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

[#1253] bugfix(jdbc): fix jdbc catalog store empty audit info issue #1261

Conversation

github-actions[bot]
Copy link

What changes were proposed in this pull request?

Mysql&Postgres catalog store empty audit info fix.

Why are the changes needed?

Fix: #1253

Does this PR introduce any user-facing change?

Users will receive audit information

How was this patch tested?

IT

…1257)

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

Mysql&Postgres catalog store empty audit info fix.

### Why are the changes needed?
Fix: #1253 

### Does this PR introduce _any_ user-facing change?
Users will receive audit information

### How was this patch tested?
IT

---------

Co-authored-by: Clearvive <clearvive@datastrato.com>
@github-actions github-actions bot added bug Something isn't working cherry-pick need backport Issues that need to backport to another branch labels Dec 26, 2023
@github-actions github-actions bot requested a review from jerryshao December 26, 2023 11:26
@jerryshao jerryshao closed this Dec 26, 2023
@jerryshao jerryshao reopened this Dec 26, 2023
Copy link
Author

Code Coverage Report

Overall Project 66.02% -0.01% 🟢
Files changed 77.14% 🟢

Module Coverage
core 76.24% 🟢
catalog-jdbc-common 39.31% -0.48% 🔴
Files
Module File Coverage
core StringIdentifier.java 99.38% 🟢
CatalogOperationDispatcher.java 88.75% 🟢
BaseMetalake.java 79.46% 🟢
MetalakeManager.java 78.77% 🟢
CatalogManager.java 62.45% 🟢
catalog-jdbc-common JdbcCatalogOperations.java 0% -1.35% 🔴

Copy link
Contributor

@qqqttt123 qqqttt123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@qqqttt123 qqqttt123 merged commit 4931534 into branch-0.3 Dec 26, 2023
4 checks passed
@qqqttt123 qqqttt123 deleted the cherry-pick-branch-0.3-06f8d47c5aa2e0d0c765a54ddb8a9978b68a31c3 branch December 26, 2023 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cherry-pick need backport Issues that need to backport to another branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants