seatunnel icon indicating copy to clipboard operation
seatunnel copied to clipboard

[Improve][e2e]change the jdbc driver scope to provider

Open liugddx opened this issue 3 years ago • 4 comments

Purpose of this pull request

change the jdbc driver scope to provider

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 17 '22 14:09 liugddx

subtask of #2642 .change the jdbc driver scope to provider and improve the jdbc connector doc. PTAL @ashulin @Hisoka-X @TyrantLucifer thanks

liugddx avatar Sep 18 '22 07:09 liugddx

PTAL @hailin0

liugddx avatar Sep 20 '22 14:09 liugddx

@ashulin i fixed those problems, and PTAL again ,thanks.

liugddx avatar Sep 21 '22 02:09 liugddx

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

liugddx avatar Sep 22 '22 02:09 liugddx

@TyrantLucifer @EricJoy2048 PTAL thanks

liugddx avatar Sep 24 '22 13:09 liugddx

Need to add driver package download for all JDBC inheritance tests E.g XaGroupOpsImplIT.testRecoverAndRollback

ic4y avatar Sep 26 '22 01:09 ic4y

Done. @ic4y .please rerun CI image @EricJoy2048 thanks.

liugddx avatar Sep 26 '22 03:09 liugddx

please rerun CI. @EricJoy2048

liugddx avatar Sep 28 '22 01:09 liugddx

@TyrantLucifer PTAL

liugddx avatar Sep 28 '22 02:09 liugddx