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

Source #12

Merged
merged 36 commits into from
Feb 8, 2023
Merged

Source #12

merged 36 commits into from
Feb 8, 2023

Conversation

maksenius
Copy link
Contributor

@maksenius maksenius commented Jan 26, 2023 •
edited
Loading

Description

add source connector

Quick checks:

  • I have followed the Code Guidelines.
  • 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.

@maksenius
add general config
2a0a644
@maksenius
Merge pull request #1 from conduitio-labs/general/config
f7ae992 
add general config
@maksenius
add destination
ecd46a6
@maksenius
Merge pull request #2 from conduitio-labs/destination
07aef7b 
add destination
@maksenius
add column types conversion
56cd656
@maksenius
Merge pull request #3 from conduitio-labs/feature/destinaton/columntypes
e0db11c 
add column types conversion
@maksenius
fix make file
9649fb8
@maksenius
fix validation
ac35eb9
@maksenius
add unit tests
c2982e2
@maksenius
Merge pull request #5 from conduitio-labs/feature/destination-test
765d80c 
add unit tests
@maksenius
add sdk validation
90679eb
@maksenius
add sdk validation
ae63470
@maksenius
add source struct
d8e1428
@maksenius
update validation
2d1c7ff
@maksenius
add snapshot logic
9d77aea
@maksenius
add source test
d7d6f76
@maksenius
implement cdc setup logic
274045e
@maksenius
combine iterator logic
b961b0a
@maksenius
Merge pull request #7 from conduitio-labs/source
5ccd973 
Source
@maksenius
add suffix name
47eb964
@maksenius
Merge pull request #8 from conduitio-labs/feature/suffixname
05fe926 
add suffix name
@maksenius
add test and docs
e64fc12
@maksenius
Merge pull request #11 from conduitio-labs/feature/integration_test
417d00c 
add test and docs
@maksenius
update readme
a93c2aa
@maksenius
add source connector
ee6de47
@maksenius
Merge branch 'main' of github.com:conduitio-labs/conduit-connector-sa…
c67697b 
…p-hana into source

� Conflicts:
�	go.mod
@maksenius
update validation rule
c83c995
@maksenius
fix table description
c980393
@maksenius
fix problem has next function on cdc switching
4ee3909
@maksenius
fix comments
36b6ec4
hariso
hariso reviewed Feb 3, 2023
View reviewed changes
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
hariso
hariso reviewed Feb 3, 2023
View reviewed changes
source/position/error.go Show resolved Hide resolved
source/position/position.go Show resolved Hide resolved
source/source_test.go Outdated Show resolved Hide resolved
source/iterator/iterator.go Show resolved Hide resolved
helper/db.go Outdated Show resolved Hide resolved
columntypes/columntypes.go Show resolved Hide resolved
columntypes/columntypes.go Outdated Show resolved Hide resolved
columntypes/columntypes.go Outdated Show resolved Hide resolved
columntypes/columntypes.go Outdated Show resolved Hide resolved
config/errors.go Outdated Show resolved Hide resolved
@maksenius maksenius force-pushed the source branch 2 times, most recently from eab80da to bb93f2c Compare February 6, 2023 10:26
hariso
hariso reviewed Feb 6, 2023
View reviewed changes
README.md Outdated Show resolved Hide resolved
source/iterator/snapshot.go Show resolved Hide resolved
source/iterator/snapshot.go Show resolved Hide resolved
source/iterator/snapshot.go Show resolved Hide resolved
source/iterator/snapshot.go Show resolved Hide resolved
helper/db.go Outdated Show resolved Hide resolved
source/iterator/iterator.go Show resolved Hide resolved
source/position/position.go Outdated Show resolved Hide resolved
source/iterator/cdc.go Show resolved Hide resolved
source/iterator/cdc.go Outdated Show resolved Hide resolved
@maksenius maksenius force-pushed the source branch 7 times, most recently from 2fecdf9 to 1462c07 Compare February 7, 2023 17:50
@hariso hariso mentioned this pull request Feb 8, 2023
Closed
hariso
hariso approved these changes Feb 8, 2023
View reviewed changes
Copy link
Contributor

@hariso hariso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! 🥇

@maksenius maksenius merged commit 0acb987 into main Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hariso hariso hariso approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maksenius @hariso