public inbox for [email protected]  

help / color / mirror / Atom feed

From: Bharath Rupireddy <[email protected]>
To: Michael Paquier <[email protected]>
Cc: Jelte Fennema-Nio <[email protected]>
Cc: Aleksander Alekseev <[email protected]>
Cc: Pgsql Hackers <[email protected]>
Cc: Fabrízio de Royes Mello <[email protected]>
Subject: Re: Add test module for Table Access Method
Date: Tue, 16 Jan 2024 10:45:25 +0530
Message-ID: <CALj2ACWYQbU4_6UA5p24Vc5UKnp11sFh0E7qefqcJX2RaE-Rnw@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <CAFcNs+pcU2ib=jvjNZNboD+M2tHO+vD77C_YZJ2rsGR0Tp35mg@mail.gmail.com>
	<[email protected]>
	<CAGECzQR0OExsrx33nBkySPkDShSn-izcz+JhVrqTrJwRfAQm6w@mail.gmail.com>
	<CAJ7c6TM3=tnDYAWyyvRB6UdeoafXaPPMq+XpafW+ZBcOnkg1gA@mail.gmail.com>
	<CAGECzQQwNZ2sKh+Qfx1X+=ODEfx+9V0ig6CsmBwOOs2CAy+kmQ@mail.gmail.com>
	<[email protected]>

On Tue, Jan 16, 2024 at 10:28 AM Michael Paquier <[email protected]> wrote:
>
> Hmm.  I'd rather have it do something useful in terms of test coverage
> rather than being just an empty skull.
>
> How about adding the same kind of coverage as dummy_index_am with a
> couple of reloptions then?  That can serve as a point of reference
> when a table AM needs a few custom options.  A second idea would be to
> show how to use toast relations when implementing your new AM, where a
> toast table could be created even in cases where we did not want one
> with heap, when it comes to size limitations with char and/or varchar,
> and that makes for a simpler needs_toast_table callback.

I think a test module for a table AM will really help developers. Just
to add to the above list - how about the table AM implementing a
simple in-memory (columnar if possible) database storing tables
in-memory and subsequently providing readers with the access to the
tables?

--
Bharath Rupireddy
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com

view thread (6+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Add test module for Table Access Method
  In-Reply-To: <CALj2ACWYQbU4_6UA5p24Vc5UKnp11sFh0E7qefqcJX2RaE-Rnw@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox