FANNG1 commented on code in PR #5944:
URL: https://github.com/apache/gravitino/pull/5944#discussion_r1895619541
##########
core/src/main/java/org/apache/gravitino/listener/api/event/AlterTagFailureEvent.java:
##########
@@ -7,7 +7,7 @@ public class AlterTagFailureEvent extends TagFailureEvent {
private final String name;
private final TagChange[] changes;
public AlterTagFailureEvent(String user, String metalake, String name,
TagChange[] changes, Exception exception) {
- super(user, exception);
+ super(user, null, exception);
Review Comment:
please provide a meaning identifier to all failure event, for tag operations
the resource identifier is the $metalake.$tagName, for list operations the the
resource identifier is the objects which supports list tags, such as
$metalake.$metadataObject in `listTagsForMetadataObject`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@gravitino.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
