pvary commented on pull request #2465:
URL: https://github.com/apache/iceberg/pull/2465#issuecomment-829117889


   > Also @pvary since hive module uses schema `toJson` indicated in the 
comment below.
   
   Thanks @yyanyy for the notification.
   
   Checked the Hive related codepaths and I do not see any issue with the new 
Id which are not covered by the tests.
   We do not persist the schema json. We use the `toJson` to serialize the 
schema during the create table call, which should be ok I think.
   
   


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to