jarredhj0214 commented on PR #12482: URL: https://github.com/apache/gravitino/pull/12482#issuecomment-5338704641
> After investigating more, > we can create a new exception class `TableForbiddenException` to extend `TableNotExsitException`. It may be better. > I don't whether it works. Welcome to discuss further. Yes, the original goal of this PR is to fix the misleading Flink connector behavior where a missing table privilege is finally reported as "table not found". I agree that we should avoid changing the server-side 403/404 semantics if that may leak metadata existence. So I think the `TableForbiddenException extends TableNotExistException` approach can fit this goal: it keeps Calcite's speculative probe fallback because it is still a TableNotExistException, while allowing the final Flink error message to mention that the table may not exist or the user may not have privileges. I'll update the PR in this direction. -- 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]
