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.

Sign up for GitHub

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

Jump to bottom

[Bug report] Mysql&Postgres catalog store empty audit info #1253

Closed
qqqttt123 opened this issue Dec 25, 2023 · 1 comment · Fixed by #1257
Closed

[Bug report] Mysql&Postgres catalog store empty audit info #1253

qqqttt123 opened this issue Dec 25, 2023 · 1 comment · Fixed by #1257
Assignees
@Clearvive
Milestone
Gravitino 0.4.0

Comments

@qqqttt123
Copy link
Contributor

qqqttt123 commented Dec 25, 2023 •
edited by Clearvive
Loading

Describe what's wrong

Mysql&Postgres catalog store empty audit info

Error message and/or stacktrace

Mysql&Postgres catalog store empty audit info

How to reproduce

Mysql&Postgres catalog store empty audit info

Additional context

return new JdbcTable.Builder()
    .withName(tableName)
    .withColumns(jdbcColumns.toArray(new JdbcColumn[0]))
    .withComment(remove)
    .withProperties(properties)
    .withAuditInfo(AuditInfo.EMPTY)
    .build();
@qqqttt123
Copy link
Contributor Author

cc @Clearvive Could you help me take a look?

@Clearvive Clearvive self-assigned this Dec 26, 2023
@Clearvive Clearvive added this to the Gravitino 0.4.0 milestone Dec 26, 2023
Clearvive pushed a commit to Clearvive/gravitino that referenced this issue Dec 26, 2023
@Clearvive
[apache#1253] bugfix(jdbc): jdbc catalog store empty audit info fix.
4c91be3
@Clearvive Clearvive changed the title [Bug report] Mysql catalog store empty audit info [Bug report] Mysql&Postgres catalog store empty audit info Dec 26, 2023
@Clearvive Clearvive mentioned this issue Dec 26, 2023
Merged
Clearvive pushed a commit to Clearvive/gravitino that referenced this issue Dec 26, 2023
@Clearvive
[apache#1253] bugfix(jdbc): jdbc catalog store empty audit info fix.
3ed9e35
Clearvive pushed a commit to Clearvive/gravitino that referenced this issue Dec 26, 2023
@Clearvive
[apache#1253] bugfix(jdbc): jdbc catalog store empty audit info fix.
c8c2aad
Clearvive pushed a commit to Clearvive/gravitino that referenced this issue Dec 26, 2023
@Clearvive
[apache#1253] bugfix(jdbc): jdbc catalog store empty audit info fix.
795b014
Clearvive pushed a commit to Clearvive/gravitino that referenced this issue Dec 26, 2023
@Clearvive
[apache#1253] bugfix(jdbc): jdbc catalog store empty audit info fix.
2773e8e
jerryshao pushed a commit that referenced this issue Dec 26, 2023
@Clearvive
[#1253] bugfix(jdbc): fix jdbc catalog store empty audit info issue (#…
06f8d47 
…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 mentioned this issue Dec 26, 2023
Merged
github-actions bot pushed a commit that referenced this issue Dec 26, 2023
@Clearvive @web-flow
[#1253] bugfix(jdbc): fix jdbc catalog store empty audit info issue (#…
169f6a2 
…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>
qqqttt123 pushed a commit that referenced this issue Dec 26, 2023
@github-actions @Clearvive
[#1253] bugfix(jdbc): fix jdbc catalog store empty audit info issue (#…
4931534 
…1261)

### 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 <143773256+Clearvive@users.noreply.github.com>
Co-authored-by: Clearvive <clearvive@datastrato.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Clearvive Clearvive

Labels
None yet
Projects
None yet
Milestone
Gravitino 0.4.0
Development

Successfully merging a pull request may close this issue.

[#1253] bugfix(jdbc): fix jdbc catalog store empty audit info issue Clearvive/gravitino
2 participants
@Clearvive @qqqttt123