You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2021/01/04 15:06:24 UTC

[GitHub] [iceberg] pvary commented on pull request #1920: Hive: Serialize metadata location so split generation does not need to load the table

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


   > This looks good to me. One last thing to consider is whether we should do this in Hive or whether we should do this through serialization. If we do it in Hive, then we have code that touches `BaseTable` and does substitution. That's okay, but not great.
   
   Thanks for the review @rdblue!
   Happy new year!
   
   Went for the Serialization solution - seems like the most generic way to solve this. Thanks for the suggestion!


----------------------------------------------------------------
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:
users@infra.apache.org



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