-
Notifications
You must be signed in to change notification settings - Fork 164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add datafusion #14
add datafusion #14
Conversation
Thank you! I'm very interested in this. |
I tried to run it on c6a.4xlarge
|
@waitingkuo Let's submit these results and then improve? |
@alexey-milovidov we haven't fixed some issues in our master branch, i'll modify the installation part to build the latest and then submit it |
@qoega thank you! |
Co-authored-by: Ilya Yatsishin <[email protected]>
@qoega i fixed some issues and all the test cases passed 😃 @alexey-milovidov it's ready to merge, thank you~ |
close apache/datafusion#2902 and apache/datafusion#3048 |
Current version
|
@qoega thank you unfortunately, i just found a issue here. i'll make it right again as soon as possible |
No problem. I just have that instance ready for benchmarks and can pull and run anytime. |
@qoega @alexey-milovidov i submitted an issues for parquet data #18
|
datafusion parquet importer doesn't support schema for now, it's inferred from parquet metadata directly. |
i did the benchmark again. it works now. I thought that there were some issues it turned out that i used hits_0.parquet to do the quick test. |
@qoega @alexey-milovidov i've made the result up to date. please let me if there's anything i need to improve. thanks~ |
@waitingkuo Thank you! |
Add datafusion
not yet find a "c6a.4xlarge, 500gb gp2" vm to test this