brin_summarize_new_values error checking - Mailing list pgsql-hackers

From Jeff Janes
Subject brin_summarize_new_values error checking
Date
Msg-id CAMkU=1xUA-gDK_aF6HmDyPaDh8ENJSy_P8piG3Wxi_Gj6pHH8A@mail.gmail.com
Whole thread Raw
Responses Re: brin_summarize_new_values error checking  (Fujii Masao <masao.fujii@gmail.com>)
List pgsql-hackers
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.

Cheers,

Jeff

[1] http://www.postgresql.org/message-id/CAHGQGwH=M1BAEJPQdpJjCneqwg8Xa+P8SB+ZsvhVwH6gL2J46w@mail.gmail.com



pgsql-hackers by date:

Previous
From: Jeff Janes
Date:
Subject: Re: GIN pending list clean up exposure to SQL
Next
From: Tom Lane
Date:
Subject: Re: Add generate_series(date,date) and generate_series(date,date,integer)