Horton

Re: [WIP]Vertical Clustered Index (columnar store extension) - take2

Peter Smith <smithpb2250@gmail.com>

View thread
From: Peter Smith <smithpb2250@gmail.com>
To: "Aya Iwata (Fujitsu)" <iwata.aya@fujitsu.com>
Cc: Tomas Vondra <tomas@vondra.me>, "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Date: 2025-07-02T06:34:02Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Mark ItemPointer arguments as const throughout

  2. Reorganize GUC structs

  3. Eliminate XLOG_HEAP2_VISIBLE from vacuum phase III

  4. Update various forward declarations to use typedef

  5. Pathify RHS unique-ification for semijoin planning

  6. Revert "Don't lock partitions pruned by initial pruning"

  7. For inplace update, send nontransactional invalidations.

  8. Remove nearly-unused SizeOfIptrData macro.

On Wed, Jul 2, 2025 at 10:22 AM Aya Iwata (Fujitsu)
<iwata.aya@fujitsu.com> wrote:
>
...
>
> > reloptions.c
> > ------------------
> > - Changes that should be moved to the contrib module Why should in-core
> > reloptions know about this? See for example how "contrib/bloom" defined
> > reloptions.
>
> Thanks. We are studying how to replace this with an extensible solution
> using "contrib/bloom" as guidance. Also, as Timur (28/Jun) suggests that
> this might be unused code anyway. Investigating...

Fixed (removed) in v10.

>
> > reloptions.h
> > ------------------
> > - Why does this need to add an entry into the relopt_kind enum? It
> > should be extensible, see the "contrib/bloom".
>
> Thanks. We are studying how to replace this with an extensible
> solution using "contrib/bloom" as guidance.
> (Timur suggests that this also unused code anyway. Investigating...)

Fixed (removed) in v10.

> > rel.h
> > ------------------
>
> > - Doesn't AM have custom reloptions now? Like contrib/bloom?
>
> Thanks. We are studying how to replace this StdRdOptions change
> with an extensible solution using "contrib/bloom" as guidance.
> (Timur suggests that this also unused code anyway. Investigating...)
>

Fixed (removed) in v10.

======
Kind Regards,
Peter Smith.
Fujitsu Australia