Skip to content

Conversation

@Dandandan
Copy link
Contributor

Which issue does this PR close?

Closes #747

Rationale for this change

Upgrade to latest version, which has some performance improvements (usage of Eisel-Lemire algorithm being one important change)

See:
https://github.com/Alexhuszagh/rust-lexical/releases/tag/v0.8.0

What changes are included in this PR?

Are there any user-facing changes?

@Dandandan Dandandan changed the title Upgrade lexical-core Upgrade lexical-core to 0.8 Sep 4, 2021
@github-actions github-actions bot added the arrow Changes to the arrow crate label Sep 4, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #748 (1e9dae2) into master (3b56485) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #748   +/-   ##
=======================================
  Coverage   82.47%   82.47%           
=======================================
  Files         168      168           
  Lines       47715    47715           
=======================================
  Hits        39353    39353           
  Misses       8362     8362           
Impacted Files Coverage Δ
arrow/src/compute/kernels/boolean.rs 96.76% <ø> (ø)

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 3b56485...1e9dae2. Read the comment docs.

@Dandandan Dandandan merged commit c4e20da into apache:master Sep 5, 2021
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.

Upgrade lexical-core to 0.8

4 participants