-
-
Notifications
You must be signed in to change notification settings - Fork 4
Manage batch processing in Debezium Extensions for Quarkus #13
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
…yncEngineBuilderFactory Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
…events Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
Signed-off-by: kmos <[email protected]>
| <dependency> | ||
| <groupId>org.apache.kafka</groupId> | ||
| <artifactId>connect-api</artifactId> | ||
| <scope>provided</scope> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it really provided?
|
|
||
| import org.apache.kafka.connect.source.SourceRecord; | ||
|
|
||
| public interface BatchEvent { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really want to operate at this level? Can't we extend CapturingEvent class to add commit or similar methods.
| public interface BatchEvent { | |
| public interface ComittingCapturingEvent extends CapturingEvent { |
| private final AtomicInteger isCapturingFilteredEvent = new AtomicInteger(0); | ||
|
|
||
| @Capturing(destination = "topic.inventory.products") | ||
| public void capture(CapturingEvents<BatchEvent> events) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't we take a bit different approach so the user will use somehing like
| public void capture(CapturingEvents<BatchEvent> events) { | |
| public void capture(List<CommittedCapturingEvent> events) { |
closes debezium/dbz#1484