Skip to content

Commit

Permalink
Disable the Parquet reader's wide lists tables GTest by default (#17120)
Browse files Browse the repository at this point in the history
This PR disables Parquet reader's wide lists table gtest by default as it takes several minutes to complete with memcheck. See the discussion on PR #17059 (this [comment](#17059 (comment))) for more context.

Authors:
  - Muhammad Haseeb (https://github.com/mhaseeb123)

Approvers:
  - David Wendt (https://github.com/davidwendt)
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #17120
  • Loading branch information
mhaseeb123 authored Oct 18, 2024
1 parent e1c9a5a commit e242dce
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion cpp/tests/io/parquet_reader_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2725,7 +2725,9 @@ TYPED_TEST(ParquetReaderPredicatePushdownTest, FilterTyped)
CUDF_TEST_EXPECT_TABLES_EQUAL(expected->view(), result_table);
}

TEST_F(ParquetReaderTest, ListsWideTable)
// The test below requires several minutes to complete with memcheck, thus it is disabled by
// default.
TEST_F(ParquetReaderTest, DISABLED_ListsWideTable)
{
auto constexpr num_rows = 2;
auto constexpr num_cols = 26'755; // for slightly over 2B keys
Expand Down

0 comments on commit e242dce

Please sign in to comment.