On Mon, Jan 25, 2016 at 4:03 PM, Jeff Janes <jeff.ja...@gmail.com> wrote: > In reviewing one of my patches[1], Fujii-san has pointed out that I > didn't include checks for being in recovery, or for working on another > backend's temporary index. > > I think that brin_summarize_new_values in 9.5.0 commits those same > sins. In its case, I don't think those are critical, as they just > result in getting less specific error messages that one might hope > for, rather than something worse like a panic. > > But still, we might want to address them.
Agreed to add those checks. Also, I think we should document that index maintenance functions cannot be executed during recovery. Regards, -- Fujii Masao -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
