Re: [HACKERS] brin_summarize_new_values error checking

2017-09-23 Thread Alvaro Herrera
Fujii Masao wrote: > On Wed, Jan 27, 2016 at 11:56 AM, Fujii Masao wrote: > > On Mon, Jan 25, 2016 at 4:03 PM, Jeff Janes 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 temp

Re: [HACKERS] brin_summarize_new_values error checking

2016-04-19 Thread Fujii Masao
On Wed, Jan 27, 2016 at 11:56 AM, Fujii Masao wrote: > On Mon, Jan 25, 2016 at 4:03 PM, Jeff Janes 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 th

Re: [HACKERS] brin_summarize_new_values error checking

2016-01-26 Thread Fujii Masao
On Mon, Jan 25, 2016 at 4:03 PM, Jeff Janes 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 > si