fix: Fix Column Type Name for PostgreSQL ARRAY types

[sakthivelmanii]

PostgreSQL usually returns _ as column name for ARRAY types

[olavloite]

Can we add a test that verifies a type name in 'plain text'. That is: I suspect that the current implementation will return something like _varchar[] as the type name for a PostgreSQL array of varchar. That would be incorrect, as the correct name would be either _varchar or varchar[].

[sakthivelmanii]

now it will return _varchar. Maybe I will update the test to not to use TEST_COLUMNS but have two examples, only for ARRAY, REGULAR TYPE..

[olavloite]

Should we fix this for more than only PostgreSQL arrays? The current implementation will (I think)

1. Return ARRAY for all GoogleSQL arrays
2. STRING for PostgreSQL varchar columns (and so forth for each non-array type)

[sakthivelmanii]

The current implementation will (I think)
Return ARRAY for all GoogleSQL arrays
STRING for PostgreSQL varchar columns (and so forth for each non-array type)

Yes

Should we fix this for more than only PostgreSQL arrays?

Sorry? What do you mean by this?

[olavloite]

What I mean is that it is a bit weird that:

1. GoogleSQL ARRAY<STRING> returns ARRAY, while PostgreSQL ARRAY<STRING> returns _varchar
2. PostgreSQL STRING returns STRING (instead of varchar), while PostgreSQL ARRAY<STRING> returns _varchar

[sakthivelmanii]

I see. we can change it. I don't think there will be a problem if we change it. So far ARRAY is the only thing which was concerning for me so I changed it.. Right now, there's a complex handling of types in PG. text == varchar == charater varying. We need to maintain one type which will be easy.

If you feel, we should do it, I can incorporate the change.