seatunnel icon indicating copy to clipboard operation
seatunnel copied to clipboard

Published 20 hours ago • verified publisher icon apache

[Feature][Connector-V2] add sqlserver connector

Open liugddx opened this issue 3 years ago • 8 comments

Purpose of this pull request

support sqlserver connector

Check list

  • [x] Code changed are covered with tests, or it does not need tests for reason:
  • [x] If any new Jar binary package adding in your PR, please add License Notice according New License Guide
  • [x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

liugddx avatar Sep 05 '22 04:09 liugddx

是否应该是jdbc-connent

githublaohu avatar Sep 05 '22 18:09 githublaohu

PTAL @ic4y @EricJoy2048 @CalvinKirs thanks.

liugddx avatar Sep 18 '22 11:09 liugddx

PTAL @ic4y @EricJoy2048 @CalvinKirs thanks.

e2e test pass image

liugddx avatar Sep 18 '22 11:09 liugddx

LGTM @Hisoka-X

ic4y avatar Sep 20 '22 03:09 ic4y

PTAL @CalvinKirs @TyrantLucifer @EricJoy2048 .thanks

liugddx avatar Sep 20 '22 10:09 liugddx

@Hisoka-X @ashulin @TyrantLucifer @ic4y change /tmp/spark/seatunnel/ to /tmp/seatunnel/ PTAL ,thanks.

liugddx avatar Sep 21 '22 03:09 liugddx

@TyrantLucifer please rerun the CI. image

liugddx avatar Sep 21 '22 10:09 liugddx

@ashulin @TyrantLucifer @CalvinKirs @Hisoka-X @EricJoy2048 PTAL thanks.

liugddx avatar Sep 22 '22 02:09 liugddx

PTAL @CalvinKirs @TyrantLucifer @Hisoka-X .thanks

liugddx avatar Sep 24 '22 13:09 liugddx

LGTM

hailin0 avatar Sep 29 '22 03:09 hailin0

please rerun CI thanks @TyrantLucifer

liugddx avatar Sep 29 '22 05:09 liugddx

please rerun CI thanks @TyrantLucifer

Wait until the running workflow is finished before restarting. Please be patient.

TyrantLucifer avatar Sep 29 '22 06:09 TyrantLucifer

@TyrantLucifer @Hisoka-X PTAL

liugddx avatar Sep 29 '22 14:09 liugddx

rerun CI? I don't think this mistake affects @TyrantLucifer

liugddx avatar Sep 30 '22 10:09 liugddx

rerun CI? I don't think this mistake affects @TyrantLucifer

CI had been optimized, so some steps will be enabled, before your pr is merged CI must run successfully. It's a community norm.

image

There are some problems with the dependency checking step, pls check again.

TyrantLucifer avatar Sep 30 '22 10:09 TyrantLucifer

rerun CI? I don't think this mistake affects @TyrantLucifer

CI had been optimized, so some steps will be enabled, before your pr is merged CI must run successfully. It's a community norm.

image

There are some problems with the dependency checking step, pls check again.

At now the dependency have some problem because st-engine bring some dependency but not update LICENSE file, we will update it later.

Hisoka-X avatar Sep 30 '22 10:09 Hisoka-X

rerun CI? I don't think this mistake affects @TyrantLucifer

CI had been optimized, so some steps will be enabled, before your pr is merged CI must run successfully. It's a community norm.

image

There are some problems with the dependency checking step, pls check again.

At now the dependency have some problem because st-engine bring some dependency but not update LICENSE file, we will update it later.

Hisoka-X avatar Sep 30 '22 10:09 Hisoka-X

rerun CI? I don't think this mistake affects @TyrantLucifer

CI had been optimized, so some steps will be enabled, before your pr is merged CI must run successfully. It's a community norm. image There are some problems with the dependency checking step, pls check again.

At now the dependency have some problem because st-engine bring some dependency but not update LICENSE file, we will update it later.

Good, let's waiting other steps of CI. I had restarted the failed steps

TyrantLucifer avatar Sep 30 '22 11:09 TyrantLucifer