Design and implement a caching strategy

In attempting to get specific views max-age time values working, I discovered that Viewfield sets 'max-age' => 0 on every content type using a viewfield, no matter whether an actual view was rendered or not. This caused all content types using a Viewfield to become non-cacheable when relying on cacheable metadata.

Suggest removing the cache max-age entirely, as it appears that $view->buildRenderable($display_id, $arguments) is handling the cache settings on #content, or at least moving the max-age setting to be within the if (!empty($view->result) || $always_build_output) section to avoid having it set all of the time, even when there is no rendered view.