Skip to content

Conversation

@vincentsarago
Copy link
Member

closes #67

This PR replace string_to_table(...) by unnest(string_to_array(...)) to support older postgres

string_to_table() provides the set-based equivalent to string_to_array(), and is functionally the same albeit somewhat faster than unnest(string_to_array())

From PG docs: https://pgpedia.info/s/string_to_table.html

Note: we could also use regexp_split_to_table https://pgpedia.info/r/regexp_split_to_table.html which seems to give the same result 🤷

@vincentsarago vincentsarago requested a review from bitner September 5, 2023 21:49
@vincentsarago
Copy link
Member Author

👋 @bitner ☝️

@vincentsarago vincentsarago merged commit 627a257 into main Oct 2, 2023
@vincentsarago vincentsarago deleted the issue67 branch October 2, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

string_to_table fails with Postgres < 14

3 participants