You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
facebookincubator/velox#9948 was merged. We'd change to the new build method to make Velox build up a single libvelox.a which can be easily cached among different CI runs as long as Velox's code doesn't get changed.
A possible issue is libvelox.a probably collides with Velox backend's libvelox.so more or less so we may consider changing name for the latter.
In most cases that Velox's code doesn't change, we expect the enhancement could save up to 1 hour during running CI, for a PR that changes Gluten's C++ code so requires to build C++ library on GitHub runner.
The text was updated successfully, but these errors were encountered:
Description
facebookincubator/velox#9948 was merged. We'd change to the new build method to make Velox build up a single
libvelox.a
which can be easily cached among different CI runs as long as Velox's code doesn't get changed.A possible issue is
libvelox.a
probably collides with Velox backend'slibvelox.so
more or less so we may consider changing name for the latter.In most cases that Velox's code doesn't change, we expect the enhancement could save up to 1 hour during running CI, for a PR that changes Gluten's C++ code so requires to build C++ library on GitHub runner.
The text was updated successfully, but these errors were encountered: