[rocketmq-replicator] Support create producer with connector's params…#463
[rocketmq-replicator] Support create producer with connector's params…#463casuallc wants to merge 2 commits intoapache:masterfrom
Conversation
# Conflicts: # connectors/rocketmq-replicator/src/main/java/org/apache/rocketmq/replicator/ReplicatorSourceTask.java
|
@odbozhou Please take a look. |
|
In my opinion, this pr has some conflicts with the design of connect
|
1 similar comment
|
In my opinion, this pr has some conflicts with the design of connect
|
|
The connector is depolyed alone , so we should not bind to only one rocketmq cluster. And then what's the meaning to config |
The purpose of dest.endpoint is to allow the replicator to connect itself to the current cluster, so that the ability to automatically create a target topic or automatically create a consumerGroup can be realized. In addition to the points mentioned above, it is also very important that the consumer side can use its perfect offset management mechanism to easily achieve at least one transmission. Because the producer logic is relatively simple, it can achieve at least one transmission in scenarios with poor network conditions. A transfer is more difficult. This is why replicator uses source to implement instead of sink. |
What is the purpose of the change
In source task, producer is create with default config in
connect-standalone.conf.This commit support use connector's config to create producer.
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice,
it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyleto make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.