incubator-gluten icon indicating copy to clipboard operation
incubator-gluten copied to clipboard

[DOC] build arrow first when build Velox separately.

Open j7nhai opened this issue 1 year ago • 3 comments

What changes were proposed in this pull request?

Should build arrow first when build Velox separately.

How was this patch tested?

manually

j7nhai avatar Jul 02 '24 06:07 j7nhai

Thanks for opening a pull request!

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

https://github.com/apache/incubator-gluten/issues

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

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

github-actions[bot] avatar Jul 02 '24 06:07 github-actions[bot]

@j7nhai, could you let me know the build error you meet? I think building velox doesn't require builing arrow must be done firstly.

PHILO-HE avatar Jul 03 '24 04:07 PHILO-HE

What's the script when you build Velox? Did you apply Gluten's velox patch when you build velox?

FelixYBW avatar Jul 03 '24 05:07 FelixYBW

Actually, build Velox script includes building Arrow. The error I encountered before was that the arrow jar was not installed in the system. Maybe it was accidental.

j7nhai avatar Jul 08 '24 02:07 j7nhai