-
Notifications
You must be signed in to change notification settings - Fork 552
WIP: add bolt backend in gluten #11261
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
base: main
Are you sure you want to change the base?
Conversation
|
Thanks for your hardwork, really appreciate it @WangGuangxin . I know Velox does certain queries where the result does not match with what vanilla spark outputs. Is this the same with Bolt? I think it would be nice to include some docs about any potential mismatches for users to be cautious about |
|
@WangGuangxin can you change the time metric of "total time of" to "time of" |
@afterincomparableyum thank you for your suggestion. We would add those documents later before this PR is merged. |
|
Can you create similar documents as Velox? here is the list of Velox docs. |
|
Does Bolt passes Spark3.2, 3.3, 3.4 and 3.5 UTs? We are going to drop 3.2 support and fixing 4.0 UTs. So Bolt may starts from 3.3 support. |
b871610 to
be4494f
Compare
Currently we mainly run UTs on spark3.5. For now most of 3.5 cound be passed, and we are fixing failed uts. We will follow the community's support for different spark versions. |
283b3f4 to
7d328ca
Compare
* add make argument for specifing Bolt's version(branch/tag) * minor fix
Refine build script

No description provided.