On 2020-Mar-16, Justin Pryzby wrote:
> Also, should we call it "is_index_clustered", since otherwise it sounds alot
> like "+get_index_clustered" (without "is"), which sounds like it takes a table
> and returns which index is clustered. That might be just as useful for some of
> these callers.
Amit's proposed name seems to match lsyscache.c usual conventions better.
> Should we use your get_index_isclustered more widely ?
Yeah, in cluster(), mark_index_clustered().
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services