wmoustafa commented on PR #9830: URL: https://github.com/apache/iceberg/pull/9830#issuecomment-5201226858
> I agree with @danielcweeks too. I don't see why we need to do a freshness check here when the purpose of this method is to return the view definition if it exists. @bennychow as discussed in the MV sync, this is fixed in the analyzer-based follow-up: `loadView` no longer throws or does any freshness check at all. It always returns the view normally, and routing to the storage table happens separately via a Catalyst analyzer rule (`ResolveMaterializedViews`) that rewrites the relation before view expansion when the MV is fresh. -- 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]
