Skip to content

rg/pg_array #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 52 commits into
base: master
Choose a base branch
from
Open

rg/pg_array #2

wants to merge 52 commits into from

Conversation

Rustik
Copy link

@Rustik Rustik commented Aug 23, 2023

attempt to add Redshift support for pg_array
what was done:

  • added support for super column on insertion and serialization on fetching row
  • added method rs_join to array_ops

driv3r and others added 30 commits October 22, 2013 11:24
Redshift does not support arrays, and dump was using pg_index.indkey
which is unsupported INT2VECTOR type.

Haven't found other workaround for now than removing the primary key
check and setting it always to false. In mean time you can set it at the
model level with `set_primary_key`.
- Redshift doesn't support TEXT type, use char or varchar only
- Redshift doesn't support RETURNING at all, so we shouldn't respond
  with true for that
Added DISTKEY and SORTKEY SQL fragments to column definition methods
Redshift does not support DROP TABLE IF EXISTS
This is instead loaded from spec/database.yml - an example
file is provided. This allows different people to run tests
on their own redshift clusters.

In addition, tests have been moved under a test-specific
schema - this needs to be created before runnning tests on
your cluster.
The latest (4.13 as of writing) version of Sequel doesn't seem
to pay any attention to supports_returning? anymore. There is
a method/opts value called disable_insert_returning that has
been added back in - just ensure that this is set all the time
on a dataset.
this prevents savepoint from firing inside transaction loop
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.

8 participants