Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[NSE-86] Merge Arrow Data Source #152

Merged
merged 20 commits into from
Mar 12, 2021
Merged

Conversation

zhouyuan
Copy link
Collaborator

@zhouyuan zhouyuan commented Mar 11, 2021

What changes were proposed in this pull request?

merging arrow data source into native sql engine project
Fixes: #86

How was this patch tested?

not tested yet

@github-actions
Copy link

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/oap-project/native-sql-engine/issues

Then could you also rename commit message and pull request title in the following format?

[NSE-${ISSUES_ID}] ${detailed message}

See also:

@zhztheplayer
Copy link
Collaborator

Thanks for finishing this, Yuan!

@zhouyuan zhouyuan changed the title Wip merge2 Wip Arrow Data Source Mar 12, 2021
@zhouyuan zhouyuan changed the title Wip Arrow Data Source [NSE-86] Merge Arrow Data Source Mar 12, 2021
@github-actions
Copy link

#86

zhouyuan and others added 4 commits March 12, 2021 09:06
@zhouyuan
Copy link
Collaborator Author

@weiting-chen @zhztheplayer here's the new merged branch

@zhztheplayer
Copy link
Collaborator

@zhouyuan Verified within local mvn build/test. Thanks.

@zhouyuan zhouyuan merged commit da6d1ee into oap-project:master Mar 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merge Arrow data source
5 participants