Skip to content

Commit aeaa6ae

Browse files
committed
Added a comment for cuCascade issue #147
1 parent fe8ebad commit aeaa6ae

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

src/include/cudf/list_offset_fixup.hpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ namespace sirius {
3232
* columns use INT32 (cudf::size_type) offsets. A LIST whose offsets child is INT64 is misread
3333
* by every list algorithm, collapsing the sublists.
3434
*
35+
* Workaround for the upstream cuCascade bug NVIDIA/cuCascade#147; remove this once the LIST
36+
* offsets are no longer promoted to INT64 during reconstruction.
37+
*
3538
* This is the single, central place where that promotion is reversed: it walks the GPU table
3639
* held by @p batch and rebuilds each LIST column (recursing into nested lists) with INT32
3740
* offsets, moving the values child and null mask (no payload copy) and reallocating only the

0 commit comments

Comments
 (0)