-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-10825: [Rust] Added support for NullArray to MutableArrayData #8851
Conversation
This is awaiting #8849 to build |
Codecov Report
@@ Coverage Diff @@
## master #8851 +/- ##
=======================================
Coverage 82.61% 82.62%
=======================================
Files 202 203 +1
Lines 50048 50065 +17
=======================================
+ Hits 41347 41364 +17
Misses 8701 8701
Continue to review full report at Codecov.
|
The full set of Rust CI tests did not run on this PR :( Can you please rebase this PR against apache/master to pick up the changes in #9056 so that they do? I apologize for the inconvenience. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I gave this a read, and ran the tests locally. Looks good to me.
Closes apache#8851 from jorgecarleitao/mutable_null Authored-by: Jorge C. Leitao <jorgecarleitao@gmail.com> Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
No description provided.