seatunnel icon indicating copy to clipboard operation
seatunnel copied to clipboard

[Feature][Connector-V2] oracle connector

Open liugddx opened this issue 3 years ago • 11 comments

Purpose of this pull request

new v2 connector for oracle

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 Aug 28 '22 10:08 liugddx

#2550

liugddx avatar Aug 28 '22 10:08 liugddx

Can you give a quick review of #2250 review? Some places have not been modified

CalvinKirs avatar Aug 28 '22 11:08 CalvinKirs

@CalvinKirs flink e2e fake2oracle is ok

liugddx avatar Aug 29 '22 13:08 liugddx

CI is error. how do i fix it?

liugddx avatar Aug 30 '22 01:08 liugddx

CI is error. how do i fix it?

maven network connection error, I rerun again.

CalvinKirs avatar Aug 30 '22 01:08 CalvinKirs

CI is error ,and the error is JdbcPhoenixIT .@CalvinKirs

liugddx avatar Aug 31 '22 02:08 liugddx

Hi, rerun, due to its large size

On 08/31/2022 @.***> wrote:

CI is error ,and the error is JdbcPhoenixIT .@CalvinKirs

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

CalvinKirs avatar Aug 31 '22 02:08 CalvinKirs

CI is not stable. @CalvinKirs

liugddx avatar Aug 31 '22 06:08 liugddx

LGTM @CalvinKirs

@CalvinKirs pls review.

liugddx avatar Sep 03 '22 02:09 liugddx

Need to merge #2648

hailin0 avatar Sep 07 '22 09:09 hailin0

Please add connector doc

ic4y avatar Sep 07 '22 10:09 ic4y

@ic4y @EricJoy2048 @Hisoka-X @hailin0 please help to review ,and rerun CI.Thanks

liugddx avatar Oct 08 '22 04:10 liugddx

Already add oracle doc. @EricJoy2048

liugddx avatar Oct 08 '22 07:10 liugddx

Please merge the dev branch to your branch and push again because https://github.com/apache/incubator-seatunnel/pull/3022

EricJoy2048 avatar Oct 08 '22 15:10 EricJoy2048

Please merge the dev branch to your branch and push again because #3022

Done.

liugddx avatar Oct 08 '22 15:10 liugddx

merge #3031. fix the e2e error. @EricJoy2048 help to rerun CI,and i need the common part of the jdbc e2e for gbase 8a,thanks

liugddx avatar Oct 09 '22 04:10 liugddx