-
Notifications
You must be signed in to change notification settings - Fork 915
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support lists of structs in row lexicographic comparator #12953
Merged
rapids-bot
merged 182 commits into
rapidsai:branch-23.06
from
ttnghia:two_tables_nested_types
May 3, 2023
Merged
Changes from all commits
Commits
Show all changes
182 commits
Select commit
Hold shift + click to select a range
ff1bc7e
Add tests
ttnghia b02abae
Complete tests
ttnghia 3f6f2f3
Disable unsupported conditions
ttnghia 3609edf
Reverse `row_operator.cu`
ttnghia 7aede42
Revert "Reverse `row_operator.cu`"
ttnghia 452f1b1
Update tests
ttnghia ce8f088
Fix offset
ttnghia 799f2ae
Change return type to `unique_ptr`
ttnghia de7437e
Adapt with changes
ttnghia 6cc4390
Update copyright year
ttnghia 4092aae
Add more variable
ttnghia bc9ecc4
Implement flattening
ttnghia c9aaa79
Merge branch 'refactor_flatten_columns' into sort_nested_types
ttnghia 284a79c
Complete implementation
ttnghia c11b6e5
Cleanup
ttnghia 6d45f8f
Fix tests
ttnghia a242dff
Fix orders
ttnghia 8e77216
Fix orders again
ttnghia 1238853
Cleanup
ttnghia ec45e32
Update tests
ttnghia 71fc858
Add variable storing auxiliary data
ttnghia c8b0634
Support lists of structs
ttnghia 4d35317
Fix copyright year
ttnghia c61beef
Fix null order
ttnghia 5fe136d
Merge branch 'branch-23.04' into refactor_flatten_columns
ttnghia 37913be
include cleanup for cudf/detail/structs/utilities.hpp
karthikeyann 5ce6f6f
Cleanup
ttnghia 32085c8
Merge branch 'branch-23.04' into refactor_flatten_columns
ttnghia fdd9b23
Fix comments
ttnghia 35b87ba
Support arbitrary nested input
ttnghia 5438336
Cleanup and add docs
ttnghia cafd1ff
Fix doxygen
ttnghia 36699db
Merge branch 'branch-23.04' into refactor_flatten_columns
ttnghia 01827e4
Fix comments
ttnghia 5f8ae83
Add comment
ttnghia 02b1619
Cleanup
ttnghia 3d91de1
Add structs-of-lists tests
ttnghia 660fa3e
Complete unit tests
ttnghia 54ffd34
Update copyright year
ttnghia 8c9bedf
Reformat
ttnghia 96e28b8
Use first rank instead of dense rank
ttnghia 4e0cf5b
Cleanup more header
ttnghia 0a39d35
Merge branch 'refactor_flatten_columns' into sort_nested_types
ttnghia cc58c8a
Merge branch 'refactor_flatten_columns' into sort_nested_types
ttnghia 2b5febf
Update doxygen
ttnghia 7390c21
Merge branch 'branch-23.04' into sort_nested_types
ttnghia 74b036b
Pass `stream` parameter to `flatten_nested_columns`
ttnghia 7e874f6
Fix docs
ttnghia 9313b4c
Fix relevant unit tests
ttnghia bd65cf5
Merge branch 'branch-23.04' into sort_nested_types
ttnghia 1cf20a2
Add sort lists of structs benchmark
ttnghia 55682f3
Add benchmark code
ttnghia d445652
Revert "Add sort lists of structs benchmark"
ttnghia 4e36b15
Fix spell
ttnghia dc9751b
Merge branch 'branch-23.04' into sort_nested_types
ttnghia 1d0d7e0
Merge branch 'branch-23.04' into sort_nested_types
ttnghia 1f8ea8f
Draft implementation
ttnghia e6c5dd2
Fix compile errors
ttnghia 98ec7d2
Fix optional access
ttnghia 66cd1bc
Minor cleanup
ttnghia e96427f
Add test
ttnghia 11f856f
Fix bug in using rank method
ttnghia f6d70aa
Merge branch 'branch-23.04' into sort_nested_types
ttnghia 87e7156
Merge remote-tracking branch 'nghia/sort_nested_types' into sort_nest…
ttnghia c6d1856
Rename variables and rewrite docs
ttnghia 3003d7e
Only allow to use `sorting_physical_element_comparator` if there is l…
ttnghia 7b21434
Fix condition
ttnghia 66e278a
Remove `noexcept`
ttnghia 032f271
Misc
ttnghia 2549a07
Merge branch 'sort_nested_types' into two_tables_nested_types
ttnghia 2fa2b69
Update docs
ttnghia 6a69fa7
Rename variables
ttnghia b291b9e
Add a helper function
ttnghia 50f29cd
Add lists-of-structs test
ttnghia 2dc2d15
Complete tests
ttnghia e818922
Add a test with equal structs
ttnghia 84813e4
Merge branch 'branch-23.04' into two_tables_nested_types
ttnghia 833d48b
Merge branch 'branch-23.04' into sort_nested_types
ttnghia 3e9136f
Merge branch 'branch-23.04' into two_tables_nested_types
ttnghia e5dc067
Remove type alias
ttnghia b449c72
Merge branch 'sort_nested_types' into two_tables_nested_types
ttnghia 0818370
Complete implementation
ttnghia 1667b27
Fix existing test
ttnghia e2cc0ab
Fix test utilities
ttnghia 64c6605
Add tests
ttnghia f75f9c5
Simplify code
ttnghia 0c8a62a
Remove redundant code
ttnghia 01acdaf
Revert all changes in `row_operators.cuh`
ttnghia 9394256
Fix typo
ttnghia 0537225
Update docs
ttnghia 6ff8ede
Add comments
ttnghia b806d14
Change docs
ttnghia 9f9031c
Merge branch 'branch-23.04' into sort_structs_of_lists
ttnghia 7c63726
Fix style
ttnghia 422f93c
Fix docs
ttnghia 70abc3c
Fix depth comparision
ttnghia 02073e1
Merge branch 'branch-23.04' into sort_structs_of_lists
ttnghia 4b03070
Merge branch 'branch-23.06' into sort_structs_of_lists
ttnghia 6baefea
Merge branch 'branch-23.06' into sort_structs_of_lists
ttnghia bcfbaca
Merge branch 'sort_nested_types' into sort_list_of_structs
ttnghia 5ab1b0f
Add `type_id` to check for element comparator
ttnghia 71585bc
Merge branch 'branch-23.06' into sort_structs_of_lists
vyasr 104aeba
Add comment
ttnghia 1b8369a
Add tests with sliced input
ttnghia 654a232
Update cpp/src/table/row_operators.cu
ttnghia 9d7921d
Optimize condition, and remove `UNKNOWN_NULL_COUNT`
ttnghia c7c69db
Merge branch 'sort_structs_of_lists' into sort_lists_of_structs
ttnghia cc44621
Remove stale implementation
ttnghia 292985b
Remove type id from element comparator
ttnghia d6991f4
Merge branch 'sort_lists_of_structs' into two_tables_nested_types
ttnghia befa5d0
TMP
ttnghia 1795dfc
Revert "TMP"
ttnghia 4eabe1d
Stop decomposing column at lists column
ttnghia 589ee83
Add test
ttnghia 2ca024c
Enable check for structs of lists
ttnghia a0151c0
Reverse changes to utilities.cpp
ttnghia 9f929b2
Fix `decompose_structs`
ttnghia 3046c83
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia e25fbfc
Improve condition for `safe_for_two_table_comparator`
ttnghia e6e087e
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 4f87848
Rename parameter
ttnghia 2cfd999
Rename variable and rewrite docs
ttnghia cdcc25a
Change error message
ttnghia 77f98c5
Fix docs for `decompose_structs`
ttnghia ecaa6c5
Rewrite check conditions
ttnghia 5a77828
Add sliced input test
ttnghia 787a6b6
Implement handling for sliced input
ttnghia f464985
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia d728b38
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 0d7fcaf
Rename variables and rewrite docs
ttnghia 6139dc2
Rewrite `transform_lists_of_structs`
ttnghia 4d24c2e
Rename variables and rewrite docs
ttnghia c5a0a98
Rewrite `has_floating_point_in_struct` and its docs
ttnghia 5240f92
Rewrite `lists_of_structs_have_floating_point`
ttnghia e9c57c7
Add check
ttnghia 3bbb1ee
Fix docs
ttnghia ee92dba
Add column order to rank computation
ttnghia d0c04b9
Rewrite docs
ttnghia 17473b4
Add tests with different null orders
ttnghia c7178b7
Add sort tests
ttnghia a07e0a2
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia b8d012d
Implement `preprocessed_id`
ttnghia 984317e
Update test
ttnghia 457f593
Fix error with tables having 0 row
ttnghia dce8a72
Add comment
ttnghia e27f9c1
MISC
ttnghia e1b8c69
Reverse verbosity level
ttnghia 2e79bed
Fix comment
ttnghia 9709738
Update docs
ttnghia 8cf9b96
Other doc fixes
ttnghia a936480
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 971dd15
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia fb17cc1
Fix style
ttnghia 6ab28fa
Change variable type
ttnghia bc955f0
Fix comments
ttnghia a3017a4
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 406db2a
Add more comment
ttnghia dc0fe20
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 3e8e530
Remove `preprocessed_id` and fix docs
ttnghia f82b3a7
Fix docs
ttnghia 1aed40d
Change comments
ttnghia f0f4b89
Rename variable
ttnghia 2f406a3
Fix tests
ttnghia e07a54e
Rename function
ttnghia 1c37a17
Fix spell
ttnghia dd94820
Fix spell
ttnghia 895e2c5
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 8a4f1af
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 3e4b7b2
Rename `*ranked_children` into `*has_ranked_children`
ttnghia eb73c75
Use enum `decompose_lists_column` instead of boolean value
ttnghia c086579
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 3a12c2d
Add comment
ttnghia 3f38b82
Fix docs
ttnghia 766127c
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia 364a312
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia b254361
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia b0ee15a
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia a44d75a
Move `constexpr` order
ttnghia 4fd2ab5
Rename fuction and extract `check_physical_element_comparator`
ttnghia f0b713d
Change unit tests
ttnghia ed89fc3
Add a complex unit test
ttnghia 5122a80
Merge branch 'branch-23.06' into two_tables_nested_types
ttnghia File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we go ahead and make this an enum?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is just exposing an API that already existed in a source file and changing this would affect other code paths I'm OK punting to a follow-up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just exposing the
rank
API indetail::
namespace. Changing this would be breaking so let's do it in some follow up PR.