Skip to content

Commit

Permalink
[VL] Fix weekly build job failure (#7163)
Browse files Browse the repository at this point in the history
  • Loading branch information
PHILO-HE authored Sep 9, 2024
1 parent cc78e17 commit 856504b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions cpp/velox/tests/VeloxShuffleWriterTest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -607,7 +607,7 @@ TEST_F(VeloxHashShuffleWriterMemoryTest, kStop) {
// Reclaim bytes to shrink partition buffer.
int64_t reclaimed = 0;
ASSERT_NOT_OK(shuffleWriter->reclaimFixedSize(2000, &reclaimed));
ASSERT(reclaimed >= 2000);
ASSERT_TRUE(reclaimed >= 2000);

// Trigger spill during stop.
ASSERT_TRUE(pool.checkEvict(pool.bytes_allocated(), [&] { ASSERT_NOT_OK(shuffleWriter->stop()); }));
Expand All @@ -630,7 +630,7 @@ TEST_F(VeloxHashShuffleWriterMemoryTest, kStopComplex) {
// Reclaim bytes to shrink partition buffer.
int64_t reclaimed = 0;
ASSERT_NOT_OK(shuffleWriter->reclaimFixedSize(2000, &reclaimed));
ASSERT(reclaimed >= 2000);
ASSERT_TRUE(reclaimed >= 2000);

// Reclaim from PartitionWriter to free cached bytes.
auto payloadSize = shuffleWriter->cachedPayloadSize();
Expand Down
2 changes: 1 addition & 1 deletion dev/package.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ cd "$GLUTEN_DIR"

# build gluten with velox backend, prompt always respond y
export PROMPT_ALWAYS_RESPOND=y
./dev/buildbundle-veloxbe.sh --build_tests=ON --build_benchmarks=ON --enable_s3=ON --enable_hdfs=ON "$@"
./dev/buildbundle-veloxbe.sh --enable_vcpkg=OFF --build_tests=ON --build_benchmarks=ON --enable_s3=ON --enable_hdfs=ON "$@"

# make thirdparty package
./dev/build-thirdparty.sh

0 comments on commit 856504b

Please sign in to comment.