[SeaTunnel][Connector-v2][Doris]Support Doris Sink.
Purpose of this pull request
Check list
- [ ] Code changed are covered with tests, or it does not need tests for reason:
- [ ] If any new Jar binary package adding in your PR, please add License Notice according New License Guide
- [ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
@xtr1993 Hi, PTAL,thx
flink
Sure, i am try to do this job at same time, but there have no avaliable docker image for doris now.
This situation (because the framework itself is not integrated) requires us to create it manually. FYI : https://www.testcontainers.org/features/creating_images/ On 09/1/2022 @.***> wrote:
flink
Sure, i am try to , but there have no avaliable docker image for doris now.
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>
This situation (because the framework itself is not integrated) requires us to create it manually. FYI : https://www.testcontainers.org/features/creating_images/ On 09/1/2022 @.> wrote: flink Sure, i am try to , but there have no avaliable docker image for doris now. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.>
Thanks, i am already try to build image by myself, and it will push to dockerhub.
Thanks, i am already try to build image by myself, and it will push to dockerhub.
Hi, are there any issues with existing doris images on docker hub? On 09/1/2022 @.***> wrote:
Thanks, i am already try to build image by myself, and it will push to dockerhub. Hi, are there any issues with existing doris images on docker hub? On 09/1/2022 @.***> wrote:
it's just use for build-env, you may see the image's tag like build-env-for-1.1.0
I test this pull request by deploy it by release package.
Can you update the title of this pr and add Sink. Because I found this is a sink connector.
@CalvinKirs @hailin0 @EricJoy2048 @jbonofre i already added e2e test for doris, plz take a look again
Need to add spark e2e test
Need to add spark e2e test
i already fix it, plz take a look again @ic4y
@hailin0 I push a commit and resolve most of comments. can you take a look again, thanks a lot.
@TyrantLucifer @EricJoy2048 Plz take a look again, if there have no any more questions, plz approve the pr, thanks a lot.
@hailin0 I reply the comments.
@hk-lrzy Please fix the CI error.
Since there has been no update for a long time, we decided to close this PR. If anyone is willing to complete it, please leave a message and let us know.
Please assign to me.