-
Notifications
You must be signed in to change notification settings - Fork 335
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
[#5477] improvement(cli): display audit information on Catalog, Schema, Table #5509
[#5477] improvement(cli): display audit information on Catalog, Schema, Table #5509
Conversation
It could potentially conflict with this PR (#5476). |
clients/cli/src/main/java/org/apache/gravitino/cli/commands/AuditCommand.java
Show resolved
Hide resolved
clients/cli/src/main/java/org/apache/gravitino/cli/commands/AuditCommand.java
Outdated
Show resolved
Hide resolved
clients/cli/src/main/java/org/apache/gravitino/cli/commands/CatalogAudit.java
Show resolved
Hide resolved
Thank you for the PR. It looks good to me; just a couple of minor things to fix. |
We've merged a couple of the CLI branches, which is why there are some conflicts here. If you could fix them that would be great. If you need any help just ask. |
d9d33d5
to
d85c4dd
Compare
Done! |
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.
Thanks for you contribution
What changes were proposed in this pull request?
Add the
--audit
option to display audit information on Catalog, Schema, TableWhy are the changes needed?
This change allows users to retrieve additional audit information on Catalog/Schema/Table, providing more insights.
Close: #5477
Does this PR introduce any user-facing change?
Yes, it adds the
--audit
option toCommandEntities.CATALOG/SCHEMA/TABLE
.How was this patch tested?
To test, use a command like the following:
Check that the output matches the expected audit information.