avamingli commented on PR #702: URL: https://github.com/apache/cloudberry/pull/702#issuecomment-2497106586
> > @my-ship-it As we have refresh fast path at #682, but for external tables we don't know the status(always up to date in gp_maview_aux). This will make REFRESH command fail to do the real thing from external data. > > So, we should skip fast path for the views have external tables. That need catalog change to record if a view has external tables outside CBDB. > > Yes, I think it is a reasonable behavior. Thanks! Done in 273dae106985d139305fc79f7267ba8d8e8f507c, add `has_foreign` to identify a mv's Query has foreign tables. If true, we will avoid Refresh fast path no matter what the datastatus of mv is. This needs a catalog change, create a label for it. -- 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: commits-unsubscr...@cloudberry.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org For additional commands, e-mail: commits-h...@cloudberry.apache.org
