[GitHub] [iceberg] jackye1995 edited a comment on pull request #3448: AWS: Make the error message of ALTER TABLE RENAME TO compatible with the error for Glue Data Catalog

[GitBox <gi...@apache.org>]

jackye1995 edited a comment on pull request #3448:
URL: https://github.com/apache/iceberg/pull/3448#issuecomment-961584838


   @kbendick thanks for the comment, I somehow ignored this PR, my bad.
   
   I believe we have merged the correct fix you made in #3468, so we can close this one @tomtongue 
   
   For anyone with the same line of thought, there was a few attempts to add this already in the past. However, as you see the input and out format and serdes you set here are really just "hacks" to make Hive happy, they are not actually the correct information in the table, adding that would just mislead users. 
   
   By not setting it, we are trying to ask people to follow the right way to use an Iceberg catalog in the Hive engine, and Glue here is just an implementation of that Iceberg catalog interface similar to any other implementations out there. I don't want to create this backdoor for only GlueCatalog simply based on the argument that Glue is "Hive compatible", as in fact it is not Hive 2 and 3 compatible especially in the write path.
   
   


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org