Re: [WIP]Vertical Clustered Index (columnar store extension) - take2 - Mailing list pgsql-hackers

On Thu, Jul 31, 2025 at 6:20 PM Japin Li <japinli@hotmail.com> wrote:
>
...
> > 1.
> > I've identified another TRAP failure. Here are the reproduction steps:
> >
> > rm -rf demo
> > initdb -D demo
> > cat <<EOF >>demo/postgresql.auto.conf
> > shared_preload_libraries = 'vci'
> > max_worker_processes = '20'
> > EOF
> >
> > pg_ctl -D demo start
> >
> > cat <<EOF | psql postgres
> > CREATE EXTENSION vci;
> > CREATE TABLE t (id int, info text);
> > CREATE INDEX ON t USING vci (id);
> > INSERT INTO t SELECT id, md5(random()::text) FROM generate_series(1, 1000) id;
> > REINDEX TABLE t;
> > REINDEX TABLE t;
> > EOF
> >
> > The current VCI design doesn't support REINDEX, which is expected. But I've
> > discovered an unexpected issue: running REINDEX on a table a second time causes
> > an assertion failure.
> >
>
> It appears VCI doesn't support reindexing; add_reindex_index_hook() returns
> false, and reindex_index() exits without restoring the security context.
>

Thanks for your fix! This will be included in the next v16 patch set.

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