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

redshift Source #18

Merged
merged 32 commits into from
Aug 31, 2023
Merged

redshift Source #18

merged 32 commits into from
Aug 31, 2023

Conversation

maksenius
Copy link
Contributor

@maksenius maksenius commented Dec 22, 2022

Description

add source

Fixes # (issue)

Quick checks:

  • There is no other pull request for the same update/change.
  • I have written unit tests.
  • I have made sure that the PR is of reasonable size and can be easily reviewed.

* Add validation for common fields

* Add the source configuration
* Add validation for common fields

* Add the source configuration
* Implement source methods, add the iterator, add collumn types

* Add the copyExistingData configuration field

* Remove the columntypes_test.go file

* Update versions of connector template and Go (#5)

* Update versions of connector template and Go

* Update build.yaml for git

* Add new lines in the end of *.yml files, update the link to conduit-core team

* Add the general configuration (#1)

* Add validation for common fields (#2)

* Configuration (#3)

* Add validation for common fields

* Add the source configuration

* Implement source methods, add the iterator, add collumn types

* Add the copyExistingData configuration field

* Remove the columntypes_test.go file

* Split the iterator logic into two different logics

* Fix the variable of position, add a few comments, correct misspellings
* Add DB ping timeout

* Update pingTimeout constant
* Add Source documentation

* Add snapshot field to the documentation

* Add known limitation
@maksenius maksenius requested a review from a team as a code owner December 22, 2022 08:23
@maha-hajja maha-hajja changed the title Source redshift Source Apr 12, 2023
@hariso hariso enabled auto-merge (squash) August 31, 2023 14:26
@hariso hariso merged commit f475aac into main Aug 31, 2023
2 checks passed
@hariso hariso deleted the source branch August 31, 2023 14:30
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.

3 participants