Re: Is _ a supported way to create a column of array type? - Mailing list pgsql-general

From Piotr Findeisen
Subject Re: Is _ a supported way to create a column of array type?
Date
Msg-id CAMrFzbtAzyrQy0CSj8T1tCTJvFQKJ1ZYm2bJpRcFBqns4Tz5Vg@mail.gmail.com
Whole thread Raw
In response to Re: Is _ a supported way to create a column of array type?  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-general
Hi Tom,

Thanks for your response.

I think I need to provide some context for my question.
I am maintaining Presto connector to Postgres (https://github.com/prestosql/presto/tree/master/presto-postgresql).

When accessing a table in Postgres, we need to map columns' types to appropriate types in Presto.
For mapping arrays, we need to know number of array dimensions.
Currently we read this from pg_attribute.attndims and this does not work for _<type> columns.

1. is there a better way to get array dimensions for a column of array type?
2. is it possible to make pg_attribute.attndims have correct value when column is defined using _<type> form?


Best,
Piotr




On Thu, Apr 25, 2019 at 11:03 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
Piotr Findeisen <piotr.findeisen@starburstdata.com> writes:
> Internally, array types get a name in the form of `_<typename>`.

Typically, yes.

> *Can a user use `_<typename>` to define a column of array type?*

Sure ... didn't you try it?

> *Is it supported?*

Not really, because it's not guaranteed that the name looks like that.
There are various corner cases where something else would be generated,
either to avoid a collision, or because truncation is needed.

However, if you've taken the trouble to check what name actually got
assigned to the array type, it's perfectly valid to use that name.

> The reason I am asking is that e.g. int4[] and _int4 behave differently.
> Although they look the same, the have different pg_attribute.attndims.

Yeah.  Nothing really cares about attndims though ... it's vestigial.
Perhaps we should remove it someday.

                        regards, tom lane

pgsql-general by date:

Previous
From: Tom Lane
Date:
Subject: Re: Is _ a supported way to create a column of array type?
Next
From: Tom Lane
Date:
Subject: Re: Is _ a supported way to create a column of array type?