amogh-jahagirdar commented on PR #4428: URL: https://github.com/apache/iceberg/pull/4428#issuecomment-1171892371
> > @amogh-jahagirdar Is there any pending commit to this PR? I would like to start contributing for spark integration for this. > > @namratha2403 I raised #4922 for addressing [#4428 (comment)](https://github.com/apache/iceberg/pull/4428#discussion_r885895391) separately. I'll be updating that PR later today. At that point this PR would be unblocked or if we conclude not to set the main ref to the current snapshot when parsing, we would have a default implementation for refs() here. > > cc: @rdblue @rdblue Since we now are guaranteed to have a main ref when parsing metadata https://github.com/apache/iceberg/blob/dec56790be30180b24c468e88989fbd3b7bf9450/core/src/main/java/org/apache/iceberg/TableMetadataParser.java#L426 I think the code in this change can operate with that assumption. Let me know what you 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. 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]
