ARROW-10575: [Rust] Rename union.rs to be cosistent with other arrays #8656
Conversation
| @@ -121,8 +121,8 @@ pub use self::array_primitive::PrimitiveArray; | |||
| pub use self::array_string::LargeStringArray; | |||
| pub use self::array_string::StringArray; | |||
| pub use self::array_struct::StructArray; | |||
| pub use self::array_union::UnionArray; | |||
There was a problem hiding this comment.
👍
Looks like we could also do the same thing (move NullArray from null.rs to null_array.rs), though as a different PR. I think this PR is a good improvement itself.
|
The https://github.com/apache/arrow/pull/8656/checks?check_run_id=1394039681 build on travis seems to have been queued for many hours at this point. I am thinking that merging this PR anyways would be fine. Any thoughts? |
Yea, I agree. This is a pretty low risk given the nature of the PR and that all the other CI runs were fine. |
Closes apache#8656 from paddyhoran/rename-union Authored-by: Paddy Horan <paddyhoran@hotmail.com> Signed-off-by: alamb <andrew@nerdnetworks.org>
Closes apache#8656 from paddyhoran/rename-union Authored-by: Paddy Horan <paddyhoran@hotmail.com> Signed-off-by: alamb <andrew@nerdnetworks.org>
No description provided.