Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Added support to cast decimal #761

Merged
merged 2 commits into from
Jan 13, 2022
Merged

Added support to cast decimal #761

merged 2 commits into from
Jan 13, 2022

Conversation

jorgecarleitao
Copy link
Owner

  • decimal <-> int
  • decimal <-> float
  • decimal <-> decimal

@jorgecarleitao jorgecarleitao added the feature A new feature label Jan 13, 2022
@codecov
Copy link

codecov bot commented Jan 13, 2022

Codecov Report

Merging #761 (f936d83) into main (cc65b0e) will increase coverage by 0.58%.
The diff coverage is 84.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #761      +/-   ##
==========================================
+ Coverage   70.97%   71.56%   +0.58%     
==========================================
  Files         313      315       +2     
  Lines       16923    17395     +472     
==========================================
+ Hits        12011    12448     +437     
- Misses       4912     4947      +35     
Impacted Files Coverage Δ
src/compute/cast/primitive_to.rs 77.13% <61.29%> (-2.56%) ⬇️
src/compute/cast/decimal_to.rs 89.36% <89.36%> (ø)
src/compute/cast/mod.rs 94.24% <100.00%> (+0.35%) ⬆️
src/array/physical_binary.rs 97.87% <0.00%> (ø)
src/io/json/read/deserialize.rs 68.83% <0.00%> (+0.41%) ⬆️
src/array/binary/mutable.rs 81.70% <0.00%> (+1.13%) ⬆️
src/array/utf8/mutable.rs 87.80% <0.00%> (+1.97%) ⬆️
src/io/json/read/infer_schema.rs 89.36% <0.00%> (+3.78%) ⬆️
src/array/mod.rs 79.74% <0.00%> (+10.44%) ⬆️

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 cc65b0e...f936d83. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 1c6e8bd into main Jan 13, 2022
@jorgecarleitao jorgecarleitao deleted the decimal_cast branch January 13, 2022 15:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant