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

Raise error when invalid fields are passed to flight #1093

Merged
merged 1 commit into from Jun 23, 2022
Merged

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Jun 21, 2022

Related to #1092

@jorgecarleitao jorgecarleitao added the bug Something isn't working label Jun 21, 2022
@codecov
Copy link

codecov bot commented Jun 22, 2022

Codecov Report

Merging #1093 (a94d930) into main (e3a005d) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1093      +/-   ##
==========================================
- Coverage   81.07%   81.06%   -0.01%     
==========================================
  Files         367      367              
  Lines       35315    35319       +4     
==========================================
+ Hits        28632    28633       +1     
- Misses       6683     6686       +3     
Impacted Files Coverage Δ
src/io/flight/mod.rs 0.00% <0.00%> (ø)
src/array/binary/mod.rs 88.79% <0.00%> (-1.25%) ⬇️
src/io/ipc/read/schema.rs 94.56% <0.00%> (+0.31%) ⬆️
src/bitmap/immutable.rs 81.48% <0.00%> (+0.61%) ⬆️
src/io/ipc/read/reader.rs 96.18% <0.00%> (+0.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 e3a005d...a94d930. Read the comment docs.

@jorgecarleitao jorgecarleitao changed the title Fixed error in serializing batch to flight Raised error when invalid fields are passed to flight Jun 22, 2022
@jorgecarleitao jorgecarleitao changed the title Raised error when invalid fields are passed to flight Raise error when invalid fields are passed to flight Jun 22, 2022
@jorgecarleitao jorgecarleitao merged commit 2f1a223 into main Jun 23, 2022
@jorgecarleitao jorgecarleitao deleted the flight_err branch June 23, 2022 03:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant