This is an automated email from the ASF dual-hosted git repository.
hongze pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git
The following commit(s) were added to refs/heads/main by this push:
new 5c7c84ba3 [VL] CI: Fix CPP tests are not running (#6295)
5c7c84ba3 is described below
commit 5c7c84ba37d6269c339e20c221e971f332fe7333
Author: Hongze Zhang <[email protected]>
AuthorDate: Tue Jul 2 13:22:32 2024 +0800
[VL] CI: Fix CPP tests are not running (#6295)
* fixup
* fixup
* fixup
---
.github/workflows/velox_docker.yml | 8 ++++----
cpp/velox/tests/BufferOutputStreamTest.cc | 2 ++
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/.github/workflows/velox_docker.yml
b/.github/workflows/velox_docker.yml
index ded2032f4..36e81cc3a 100644
--- a/.github/workflows/velox_docker.yml
+++ b/.github/workflows/velox_docker.yml
@@ -617,6 +617,10 @@ jobs:
install_arrow_deps
./dev/builddeps-veloxbe.sh --run_setup_script=OFF
--enable_ep_cache=OFF --build_tests=ON \
--build_examples=ON --build_benchmarks=ON --build_protobuf=ON
+ - name: Gluten CPP Test
+ run: |
+ cd ./cpp/build && \
+ ctest -V
- uses: actions/upload-artifact@v2
with:
name: velox-native-lib-centos-8-${{github.sha}}
@@ -682,10 +686,6 @@ jobs:
working-directory: ${{ github.workspace }}
run: |
mkdir -p '${{ env.CCACHE_DIR }}'
- - name: Gluten CPP Test
- run: |
- cd $GITHUB_WORKSPACE/cpp/build && \
- ctest -V
- name: Prepare spark.test.home for Spark 3.2.2 (other tests)
run: |
cd $GITHUB_WORKSPACE/ && \
diff --git a/cpp/velox/tests/BufferOutputStreamTest.cc
b/cpp/velox/tests/BufferOutputStreamTest.cc
index 324d8c5e6..0e16f8c87 100644
--- a/cpp/velox/tests/BufferOutputStreamTest.cc
+++ b/cpp/velox/tests/BufferOutputStreamTest.cc
@@ -16,6 +16,7 @@
*/
#include "memory/BufferOutputStream.h"
+#include "compute/VeloxBackend.h"
#include "memory/VeloxColumnarBatch.h"
#include "velox/common/memory/ByteStream.h"
#include "velox/vector/tests/utils/VectorTestBase.h"
@@ -27,6 +28,7 @@ class BufferOutputStreamTest : public ::testing::Test, public
test::VectorTestBa
protected:
// Velox requires the mem manager to be instanced.
static void SetUpTestCase() {
+ VeloxBackend::create({});
memory::MemoryManager::testingSetInstance({});
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
