ViewRefreshTask logs a materialized-view refresh error but always returns nil. The scheduler also prints complete after a task returns an error. This makes a failed refresh look completed successfully in the task log.
I would like to make a small stability fix: preserve the errors (including which view failed), still attempt the remaining views, and only log successful completion when the task succeeds. Regression tests should cover partial failure, all-success and cancellation. Per-view freshness metadata or new endpoints can follow separately.
This is an AI-assisted investigation; code will be reviewed before a contribution is submitted.
ViewRefreshTasklogs a materialized-view refresh error but always returns nil. The scheduler also printscompleteafter a task returns an error. This makes a failed refresh look completed successfully in the task log.I would like to make a small stability fix: preserve the errors (including which view failed), still attempt the remaining views, and only log successful completion when the task succeeds. Regression tests should cover partial failure, all-success and cancellation. Per-view freshness metadata or new endpoints can follow separately.
This is an AI-assisted investigation; code will be reviewed before a contribution is submitted.