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.
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
Support lists of structs in row lexicographic comparator #12953
Changes from 169 commits
ff1bc7e
b02abae
3f6f2f3
3609edf
7aede42
452f1b1
ce8f088
799f2ae
de7437e
6cc4390
4092aae
bc9ecc4
c9aaa79
284a79c
c11b6e5
6d45f8f
a242dff
8e77216
1238853
ec45e32
71fc858
c8b0634
4d35317
c61beef
5fe136d
37913be
5ce6f6f
32085c8
fdd9b23
35b87ba
5438336
cafd1ff
36699db
01827e4
5f8ae83
02b1619
3d91de1
660fa3e
54ffd34
8c9bedf
96e28b8
4e0cf5b
0a39d35
cc58c8a
2b5febf
7390c21
74b036b
7e874f6
9313b4c
bd65cf5
1cf20a2
55682f3
d445652
4e36b15
dc9751b
1d0d7e0
1f8ea8f
e6c5dd2
98ec7d2
66cd1bc
e96427f
11f856f
f6d70aa
87e7156
c6d1856
3003d7e
7b21434
66e278a
032f271
2549a07
2fa2b69
6a69fa7
b291b9e
50f29cd
2dc2d15
e818922
84813e4
833d48b
3e9136f
e5dc067
b449c72
0818370
1667b27
e2cc0ab
64c6605
f75f9c5
0c8a62a
01acdaf
9394256
0537225
6ff8ede
b806d14
9f9031c
7c63726
422f93c
70abc3c
02073e1
4b03070
6baefea
bcfbaca
5ab1b0f
71585bc
104aeba
1b8369a
654a232
9d7921d
c7c69db
cc44621
292985b
d6991f4
befa5d0
1795dfc
4eabe1d
589ee83
2ca024c
a0151c0
9f929b2
3046c83
e25fbfc
e6e087e
4f87848
2cfd999
cdcc25a
77f98c5
ecaa6c5
5a77828
787a6b6
f464985
d728b38
0d7fcaf
6139dc2
4d24c2e
c5a0a98
5240f92
e9c57c7
3bbb1ee
ee92dba
d0c04b9
17473b4
c7178b7
a07e0a2
b8d012d
984317e
457f593
dce8a72
e27f9c1
e1b8c69
2e79bed
9709738
8cf9b96
a936480
971dd15
fb17cc1
6ab28fa
bc955f0
a3017a4
406db2a
dc0fe20
3e8e530
f82b3a7
1aed40d
f0f4b89
2f406a3
e07a54e
1c37a17
dd94820
895e2c5
8a4f1af
3e4b7b2
eb73c75
c086579
3a12c2d
3f38b82
766127c
364a312
b254361
b0ee15a
a44d75a
4fd2ab5
f0b713d
ed89fc3
5122a80
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.