[FEA] Refactor transform_lists_of_structs in row_operators.cu
#13287
Assignees
Labels
feature request
New feature or request
Comments
ttnghia
added
feature request
This was referenced May 4, 2023
rapids-bot bot
pushed a commit
that referenced
this issue
May 15, 2023
This split the functions `transform_lists_of_structs` in `row_operators.cu` into separate functions or simplified their implementation. From one function that can process both cases of having either one or two input columns, now we have two functions, each one processes one case. Closes #13287. Authors: - Nghia Truong (https://github.com/ttnghia) Approvers: - Divye Gala (https://github.com/divyegala) - Vyas Ramasubramani (https://github.com/vyasr) - Mike Wilson (https://github.com/hyperbolic2346) - Bradley Dice (https://github.com/bdice) URL: #13288
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As discussed in https://github.com/rapidsai/cudf/pull/12953/files#r1176816066, the function
transform_lists_of_structsinrow_operators.cushould be either split into two separate functions or simplified its implementation (by using #12932). Since #12932 is no longer relevant, we are left with just one option which is to split it into two separate functions, each processing one table input vs two tables input.The text was updated successfully, but these errors were encountered: