| From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
|---|---|
| To: | Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com> |
| Cc: | Andres Freund <andres(at)anarazel(dot)de>, Greg Stark <stark(at)mit(dot)edu>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Zhihong Yu <zyu(at)yugabyte(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: PATCH: Using BRIN indexes for sorted output |
| Date: | 2023-02-16 16:10:16 |
| Message-ID: | 20230216161016.GV1653@telsasoft.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Feb 16, 2023 at 03:07:59PM +0100, Tomas Vondra wrote:
> Rebased version of the patches, fixing only minor conflicts.
Per cfbot, the patch fails on 32 bit builds.
+ERROR: count mismatch: 0 != 1000
And causes warnings in mingw cross-compile.
On Sun, Oct 23, 2022 at 11:32:37PM -0500, Justin Pryzby wrote:
> I think new GUCs should be enabled during patch development.
> Maybe in a separate 0002 patch "for CI only not for commit".
> That way "make check" at least has a chance to hit that new code
> paths.
>
> Also, note that indxpath.c had the var initialized to true.
In your patch, the amstats guc is still being set to false during
startup by the guc machinery. And the tests crash everywhere if it's
set to on:
TRAP: failed Assert("(nmatches_unique >= 1) && (nmatches_unique <= unique[nvalues-1])"), File: "../src/backend/access/brin/brin_minmax.c", Line: 644, PID: 25519
> . Some typos in your other patches: "heuristics heuristics". ste.
> lest (least).
These are still present.
--
Justin
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2023-02-16 16:13:22 | Re: Refactor calculations to use instr_time |
| Previous Message | Tomas Vondra | 2023-02-16 15:50:30 | Re: logical decoding and replication of sequences, take 2 |