Skip to content

Conversation

@waynexia
Copy link
Member

Which issue does this PR close?

Closes #.

Rationale for this change

See discussion in here

What changes are included in this PR?

Fix compile error in cherry-pick #621.

Are there any user-facing changes?

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
@github-actions github-actions bot added the arrow Changes to the arrow crate label Jul 28, 2021
@waynexia waynexia mentioned this pull request Jul 28, 2021
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (cherry_pick_2113e2b1@90a4a8d). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 30eed9f differs from pull request most recent head 06d28c5. Consider uploading reports for the commit 06d28c5 to get more accurate results
Impacted file tree graph

@@                   Coverage Diff                   @@
##             cherry_pick_2113e2b1     #624   +/-   ##
=======================================================
  Coverage                        ?   82.48%           
=======================================================
  Files                           ?      167           
  Lines                           ?    46463           
  Branches                        ?        0           
=======================================================
  Hits                            ?    38327           
  Misses                          ?     8136           
  Partials                        ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 90a4a8d...06d28c5. Read the comment docs.

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @waynexia

@alamb alamb merged commit 1ea4120 into apache:cherry_pick_2113e2b1 Jul 28, 2021
alamb added a commit that referenced this pull request Jul 29, 2021
* Sort binary (#569)

* impl sort fixed binary array

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* remove array builder util, add test cases

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* impl sort for generic binary array

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* tidy

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* run clippy

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* test: fixed binary with different prefix

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* rebase master

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* Fix compile error in cherry pick 2113e2b (#624)

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

Co-authored-by: Ruihang Xia <waynestxia@gmail.com>
@waynexia waynexia deleted the fix-cherry-pick-2113eab1 branch June 10, 2022 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arrow Changes to the arrow crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants