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.

Sign up for GitHub

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

Jump to bottom

Rename destination tables #1737

Merged
merged 7 commits into from
Jan 20, 2021
Merged

Rename destination tables #1737

merged 7 commits into from
Jan 20, 2021

Conversation

ChristopheDuong
Copy link
Contributor

@ChristopheDuong ChristopheDuong commented Jan 20, 2021 •
edited
Loading

What

Closes #1721

Pre-merge Checklist

  • Run integration tests
  • Publish Docker images

@ChristopheDuong ChristopheDuong changed the base branch from master to cgardens/allow_empty_password January 20, 2021 07:42
@ChristopheDuong ChristopheDuong force-pushed the chris/rename_destination_tables branch from 5b1e4bf to 12b1391 Compare January 20, 2021 07:55
@ChristopheDuong ChristopheDuong marked this pull request as ready for review January 20, 2021 09:49
@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Jan 20, 2021 •
edited by github-actions bot
Loading

/test connector=destination-snowflake

🕑 destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/498240689
✅ destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/498240689

@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Jan 20, 2021 •
edited by github-actions bot
Loading

/test connector=destination-bigquery

🕑 destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/498240902
✅ destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/498240902

@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Jan 20, 2021 •
edited by github-actions bot
Loading

/test connector=destination-redshift

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

@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Jan 20, 2021 •
edited by github-actions bot
Loading

/test connector=destination-postgres

🕑 destination-postgres https://github.com/airbytehq/airbyte/actions/runs/498241582
✅ destination-postgres https://github.com/airbytehq/airbyte/actions/runs/498241582

@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Jan 20, 2021 •
edited by github-actions bot
Loading

/test connector=destination-csv

🕑 destination-csv https://github.com/airbytehq/airbyte/actions/runs/498242280
✅ destination-csv https://github.com/airbytehq/airbyte/actions/runs/498242280

@ChristopheDuong
Copy link
Contributor Author

ChristopheDuong commented Jan 20, 2021 •
edited by github-actions bot
Loading

/test connector=destination-local-json

🕑 destination-local-json https://github.com/airbytehq/airbyte/actions/runs/498242984
✅ destination-local-json https://github.com/airbytehq/airbyte/actions/runs/498242984

Base automatically changed from cgardens/allow_empty_password to master January 20, 2021 16:24
@ChristopheDuong ChristopheDuong merged commit ebc24f8 into master Jan 20, 2021
@ChristopheDuong ChristopheDuong deleted the chris/rename_destination_tables branch January 20, 2021 16:37
@ChristopheDuong ChristopheDuong mentioned this pull request Jan 27, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cgardens cgardens cgardens approved these changes

@michel-tricot michel-tricot Awaiting requested review from michel-tricot

Assignees
No one assigned
Labels
connectors/destination/bigquery connectors/destination/local-csv connectors/destination/local-json connectors/destination/snowflake connectors/destinations-files connectors/destinations-warehouse
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Prefix destination table names with _airbyte_raw_ instead of _raw suffix
3 participants
@ChristopheDuong @cgardens @karinakuz