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

On Wed, Jul 2, 2025 at 10:22 AM Aya Iwata (Fujitsu)
<iwata(dot)aya(at)fujitsu(dot)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