datafusion-ballista icon indicating copy to clipboard operation
datafusion-ballista copied to clipboard

CI should run Ballista integration tests

Open andygrove opened this issue 4 years ago • 4 comments

Is your feature request related to a problem or challenge? Please describe what you are trying to do. We are seeing PRs merged that cause regressions in Ballista because we currently rely on manual integration testing.

Describe the solution you'd like CI should run the integration tests:

./dev/integration-tests.sh

Describe alternatives you've considered None

Additional context None

andygrove avatar May 11 '21 23:05 andygrove

The lack of ballista coverage in CI came up again with @gaojun2048 @liukun4515 and @mingmwang on https://github.com/apache/arrow-datafusion/pull/2070#discussion_r834104400

alamb avatar Mar 24 '22 18:03 alamb

I can take this task in the Q2

liukun4515 avatar Mar 25 '22 03:03 liukun4515

Thank you @liukun4515 ! In case you missed it, the setup I had in https://github.com/apache/arrow-datafusion/pull/688 already works and was only paused pending optimizations to reduce cache size. I imagine you might be able to reuse some of the setup there when you start working on this this task.

houqp avatar Mar 28 '22 01:03 houqp

Should we consider running the integration tests in CI without creating docker images? Just run the native binaries for scheduler and executor?

andygrove avatar May 04 '22 21:05 andygrove