Date: Tue, 5 Jul 2022 14:58:38 -0500
From: Justin Pryzby <pryzby@telsasoft.com>
To: Alvaro Herrera <alvherre@alvh.no-ip.org>
Cc: Jaime Casanova <jcasanov@systemguards.com.ec>,
	Roberto Mello <roberto.mello@gmail.com>,
	pgsql-hackers@lists.postgresql.org
Subject: Re: doc: BRIN indexes and autosummarize
Message-ID: <20220705195838.GT13040@telsasoft.com>
References: <20220704212227.GN13040@telsasoft.com>
 <YsNScLPHKcBBp4oO@ahch-to>
 <20220705114727.ryirrnks7rijhdh3@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220705114727.ryirrnks7rijhdh3@alvherre.pgsql>
User-Agent: Mutt/1.9.4 (2018-02-28)
Archived-At: 
 <https://www.postgresql.org/message-id/20220705195838.GT13040%40telsasoft.com>
Precedence: bulk

On Tue, Jul 05, 2022 at 01:47:27PM +0200, Alvaro Herrera wrote:
> OK, I have adopted all your proposed changes, thanks for submitting in
> both forms.  I did some more wordsmithing and pushed, to branches 12 and
> up.  11 fails 'make check', I think for lack of Docbook id tags, and I
> didn't want to waste more time.  Kindly re-read the result and let me
> know if I left something unaddressed, or made something worse.  The
> updated text is already visible in the website:
> https://www.postgresql.org/docs/devel/brin-intro.html

One issue:

+   summarized range, the range that the new page belongs into
+   does not automatically acquire a summary tuple;

"belongs into" sounds wrong - "belongs to" is better.

I'll put that change into my "typos" branch to fix later if it's not addressed
in this thread.

-- 
Justin