Skip to content

Conversation

@hawk9821
Copy link
Contributor

@hawk9821 hawk9821 commented Nov 25, 2024

Purpose of this pull request

support read arrow data, convert arrow data to SeatunnelRow. At present, doris connector and starrocks connector use arrow mode to read data and extract this part of logic.

Does this PR introduce any user-facing change?

doris connector source , starrocks connector source

How was this patch tested?

UT: Arrow2SeatunnelRowReaderTest#testSeatunnelRow
Arrow2SeatunnelRowReaderTest#testConvertArrowSpeed

Check list

@github-actions github-actions bot added dependencies Pull requests that update a dependency file connectors-v2 api labels Nov 25, 2024
@github-actions github-actions bot removed the api label Nov 26, 2024
@hawk9821 hawk9821 force-pushed the arrow branch 3 times, most recently from ccd1b1b to 7fe4021 Compare November 28, 2024 14:10
@github-actions github-actions bot added the api label Dec 2, 2024
@hawk9821 hawk9821 force-pushed the arrow branch 2 times, most recently from 078e298 to 96ab87e Compare December 2, 2024 09:32
@hawk9821 hawk9821 force-pushed the arrow branch 4 times, most recently from 890ef6f to e56e7cb Compare December 4, 2024 00:35
Copy link
Member

@hailin0 hailin0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Hisoka-X Hisoka-X merged commit 4710ea0 into apache:dev Dec 6, 2024
6 checks passed
@hawk9821 hawk9821 deleted the arrow branch December 12, 2024 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api approved connectors-v2 dependencies Pull requests that update a dependency file reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants