Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

Simplify JSON column types #545

Merged
merged 3 commits into from
May 5, 2023
Merged

Simplify JSON column types #545

merged 3 commits into from
May 5, 2023

Conversation

nolar
Copy link
Contributor

@nolar nolar commented May 3, 2023

A quick follow-up on #543 to simplify JSON column types before it is released.

In #533, similar types will follow — Struct & Array.

But we should better not bind these changes by time.

@nolar nolar requested a review from dlawin May 3, 2023 16:19
nolar referenced this pull request in nicolasaldecoa/sqeleton May 3, 2023
…port

Add support for column types: PostgreSQL JSON/JSONB and RedShift Super
@nolar nolar changed the base branch from followup-sqeleton to master May 5, 2023 10:51
@nolar nolar force-pushed the simplify-json-columns branch from 74edbcb to c2c5102 Compare May 5, 2023 10:51
Sergey Vasilyev added 3 commits May 5, 2023 21:08
We know that all these classes are column types, there is no need to specify that.

Also to consistency with other column types.
@nolar nolar force-pushed the simplify-json-columns branch from c2c5102 to 46c8867 Compare May 5, 2023 19:08
@nolar nolar merged commit d198647 into master May 5, 2023
@nolar nolar deleted the simplify-json-columns branch May 5, 2023 19:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants