Skip to content

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
@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