roryqi commented on PR #10720: URL: https://github.com/apache/gravitino/pull/10720#issuecomment-4350760181
> I think letting users configure any delimiter they want is more flexible. Defining an allowed list will always have a problem that users want a delimiter other than in the allowed list. > > From a fresh user's perspective, I think rejection is better than inconsistent behavior between catalogs. Otherwise, they will be confused whether catalog A's behavior is different from catalog B implicitly. > > From an existing user's perspective, if they already use the delimiter, failing the operation will break their compatibility. If we can add something in the code level to let them explicitly know that this new feature will affect their current usage, I think it is acceptable. Otherwise, mentioning in the doc may not be enough. Thanks for @jerryshao's suggestion. cc @mchades -- 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]
