kbendick commented on pull request #3668: URL: https://github.com/apache/iceberg/pull/3668#issuecomment-986402769
> Glue is just trying to be generic, and the Athena doc requirement is based on Hive specification. I am trying to not remove this validation unless there is an absolute need. Using special characters cause a lot of incompatibility across different compute engines. Given the fact that people are likely going to create new namespaces for Iceberg tables, I think it's better to just advocate people to use normal characters instead of allowing too many variations. > > Please let me know what you think, or if you have some specific use case to satisfy. I would also go off of the stricter requirement. From my knowledge, Glue is a bit more general purpose from it's earliest dags. So I'm not surprised that its possibly more permissive in what it allows to be created via CLI etc. But it's unlikely that its just the Validation that's causing the hold up. With many catalogs, there are limitations on the database name (or differences in how they're represented by the catalogs TableOperations). There are also engine-specific considerations, eg Athena (or Trinodb in the general case). I'm not at AWS (nor have I ever been affiliated), so I would of course defer to @jackye1995. But wanted to provide further insight as to why the change might not be as simple as removing Precondition checks. 🙂 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
