I wrote:
> So where are we on this? Should I proceed with my patch, or are we
> going to do further investigation? Does anyone want to do an actual
> patch review?
[ crickets... ]
So I took that as license to proceed, but while doing a final round of
testing I found out that a CLOBBER_CACHE_RECURSIVELY build fails,
because now that's an infinite recursion. On reflection it's a bit
surprising that it wasn't so all along. What I'm inclined to do about
it is to adjust AcceptInvalidationMessages so that there's a finite
recursion depth limit in the CLOBBER_CACHE_RECURSIVELY case, as there
already is in the CLOBBER_CACHE_ALWAYS case. Maybe 3 or so levels
would be enough.
regards, tom lane