Feedback-ID: ia2694551:Fastmail
Date: Wed, 6 Jul 2022 09:45:31 +0200
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: Justin Pryzby <pryzby@telsasoft.com>
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: <20220706074531.oiuhmmb2oub5zuch@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20220705195838.GT13040@telsasoft.com>
Archived-At: 
 <https://www.postgresql.org/message-id/20220706074531.oiuhmmb2oub5zuch%40alvherre.pgsql>
Precedence: bulk

On 2022-Jul-05, Justin Pryzby wrote:

> 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.

Hah, and I was wondering if "belongs in" was any better.

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

Roberto has some more substantive comments on the new text, so let's try
and fix everything together.  This time, I'll let you guys come up with
a new patch.

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/