Abyss-lord commented on issue #10599: URL: https://github.com/apache/gravitino/issues/10599#issuecomment-4167133210
@justinmclean Thanks for raising this. At the moment, `ListModelFailureEvent` is following the same pattern as other `FailureEvent` implementations by using `NameIdentifier.of(namespace.levels())`. From that perspective, its current behavior is consistent with the existing event model. If we believe this representation is semantically incorrect for list failures, then I think we should address it in a consistent way across all similar `FailureEvent`s, rather than changing only `ListModelFailureEvent`. Otherwise, keeping it as is would preserve consistency with the current convention. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
