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