Jia Fan
Jia Fan
Could you add test case in e2e? Thanks.
cc @hailin0 @EricJoy2048
Good, looking forward your contribute!
Could you follow the guide to open CI on your fork repository? https://github.com/apache/seatunnel/pull/6755/checks?check_run_id=24239500575
> > Could you follow the guide to open CI on your fork repository? https://github.com/apache/seatunnel/pull/6755/checks?check_run_id=24239500575 > > Yes, it has been enabled @Hisoka-X Then push a empty commit to trigger...
I think we should add a e2e test case in https://github.com/apache/seatunnel/blob/045d3e3494eac67cef555c190ecb68638d497349/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-elasticsearch-e2e/src/test/java/org/apache/seatunnel/e2e/connector/elasticsearch/ElasticsearchIT.java#L68
Hi @CosmosNi , thanks for add test! Could you retry the test case after merge from dev?
 merge code from upstream
Thanks @CosmosNi for update, could you follow the suggestion of @EricJoy2048 to mark schema config in ES source as deprecated? Both in doc and code.