-
Notifications
You must be signed in to change notification settings - Fork 784
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
Add Rust Docs examples for UnionArray #1241
Add Rust Docs examples for UnionArray #1241
Conversation
Signed-off-by: remzi <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #1241 +/- ##
==========================================
- Coverage 82.94% 82.94% -0.01%
==========================================
Files 178 178
Lines 51420 51420
==========================================
- Hits 42651 42650 -1
- Misses 8769 8770 +1
Continue to review full report at Codecov.
|
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.
This looks good to me 👍
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.
Thank you very much for the contribution @HaoYang670
/// None, | ||
/// ).unwrap(); | ||
/// | ||
/// let value = array.value(0).as_any().downcast_ref::<Int32Array>().unwrap().value(0); |
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.
It isn't the nicest API in the world, is it ?
However, documenting what we have is 👍
Thank you @HaoYang670
thank you @HaoYang670 |
Signed-off-by: remzi [email protected]
Which issue does this PR close?
Closes #751
Rationale for this change
Add 2 docs examples of building
UnionArray
using the functionUnionArray::try_new
,so that users can easily understand the data structure of
UnionArray
.What changes are included in this PR?
Are there any user-facing changes?