Re: BRIN INDEX value

Tatsuo Ishii wrote:
> > On 9/4/2015 2:04 PM, Tatsuo Ishii wrote:
> >>> One thing I imagine we could do is to change the signature of
> >>> summrize_range() to also include heapNumBlks which its (only) caller
> >>> brinsummarize() already computes. It will look like:
> >>>
> >>> static void summarize_range(IndexInfo *indexInfo, BrinBuildState *state,
> >>>                               Relation heapRel,
> >>>                               BlockNumber heapBlk,
> >>>                               BlockNumber heapNumBlks);
> >>>
> >>> I'd think changing summarize_range()'s signature would be relatively
> >>> easier/safer.
> >> 
> >> Yeah, sounds good.
> > 
> > Here's a patch to do that.
> 
> Thanks. It looks good to me (and passed all the regression tests in
> master branch). I will commit your patch if there's no objection.

Yeah, thanks, please go ahead.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services