Skip to content
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

Missing keywords in redshift #2700

Merged
merged 2 commits into from
Apr 1, 2021

Conversation

ChristopheDuong
Copy link
Contributor

@ChristopheDuong ChristopheDuong commented Apr 1, 2021

What

Replicating Mixpanel data to Redshift fails on Normalization because that source is using special keywords that redshift does not accept (but is not listed in their docs page)

How

Add those secret reserved keywords for redshift

Pre-merge Checklist

  • Run integration tests
  • Publish Docker images

@auto-assign auto-assign bot requested review from davinchia and jrhizor April 1, 2021 09:53
@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Apr 1, 2021

/publish connector=bases/base-normalization

🕑 bases/base-normalization https://github.com/airbytehq/airbyte/actions/runs/708173846
✅ bases/base-normalization https://github.com/airbytehq/airbyte/actions/runs/708173846

@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Apr 1, 2021

/test connector=connectors/destination-redshift

🕑 connectors/destination-redshift https://github.com/airbytehq/airbyte/actions/runs/708262322
✅ connectors/destination-redshift https://github.com/airbytehq/airbyte/actions/runs/708262322

@ChristopheDuong ChristopheDuong merged commit 0b6a983 into master Apr 1, 2021
@ChristopheDuong ChristopheDuong deleted the chris/normalization-redshift-keywords branch April 1, 2021 15:33
davinchia pushed a commit that referenced this pull request Apr 6, 2021
* Missing keywords in redshift
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.

2 participants