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

Added cast date32 -> i64 and date64 -> i32 #308

Merged
merged 1 commit into from
Aug 21, 2021

Conversation

ritchie46
Copy link
Collaborator

This add casting the primitive values of datetypes, date32, date64 to integers.

@codecov
Copy link

codecov bot commented Aug 21, 2021

Codecov Report

Merging #308 (524ea7f) into main (2873e1a) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #308      +/-   ##
==========================================
+ Coverage   80.57%   80.59%   +0.01%     
==========================================
  Files         324      324              
  Lines       21221    21231      +10     
==========================================
+ Hits        17099    17111      +12     
+ Misses       4122     4120       -2     
Impacted Files Coverage Δ
src/compute/cast/mod.rs 89.31% <100.00%> (+0.08%) ⬆️
tests/it/compute/cast.rs 99.32% <100.00%> (+0.01%) ⬆️
src/io/json/read/reader.rs 75.67% <0.00%> (+1.35%) ⬆️
src/io/json_integration/write.rs 6.25% <0.00%> (+6.25%) ⬆️

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 2873e1a...524ea7f. Read the comment docs.

@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Aug 21, 2021
@jorgecarleitao jorgecarleitao changed the title add cast date32 -> i64; date64 -> i32 add cast date32 -> i64 and date64 -> i32 Aug 21, 2021
@jorgecarleitao jorgecarleitao merged commit d873730 into jorgecarleitao:main Aug 21, 2021
@jorgecarleitao jorgecarleitao changed the title add cast date32 -> i64 and date64 -> i32 Added cast date32 -> i64 and date64 -> i32 Aug 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants