This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/main by this push:
new fb775011f9 doc: remove disclaimer about `ListView` not being fully
supported (#9356)
fb775011f9 is described below
commit fb775011f9e98f7eb84c8df006f8bd9e040ec505
Author: Jeffrey Vo <[email protected]>
AuthorDate: Fri Feb 6 01:04:02 2026 +0900
doc: remove disclaimer about `ListView` not being fully supported (#9356)
# Which issue does this PR close?
<!--
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases.
You can link an issue to this PR using the GitHub syntax.
-->
- Closes #5375
# Rationale for this change
<!--
Why are you proposing this change? If this is already explained clearly
in the issue then this section is not needed.
Explaining clearly why changes are proposed helps reviewers understand
your changes and offer better suggestions for fixes.
-->
Thanks to @codephage2020 & @vegarsti we were able to get the last bit of
support for `ListViews` in place for the core arrow codebase. Whilst
some tickets are outstanding (parquet & json support), I think we can
confidently declare we support `ListView` types now and closing the
tracking issue.
# What changes are included in this PR?
<!--
There is no need to duplicate the description in the issue here but it
is sometimes worth providing a summary of the individual changes in this
PR.
-->
Remove disclaimer about `ListView`s not being fully supported.
# Are these changes tested?
<!--
We typically require tests for all PRs in order to:
1. Prevent the code from being accidentally broken by subsequent changes
2. Serve as another way to document the expected behavior of the code
If tests are not included in your PR, please explain why (for example,
are they covered by existing tests)?
-->
Doc changes.
# Are there any user-facing changes?
<!--
If there are user-facing changes then we may require documentation to be
updated before approving the PR.
If there are any breaking changes to public APIs, please call them out.
-->
Doc changes.
---
arrow-schema/src/datatype.rs | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/arrow-schema/src/datatype.rs b/arrow-schema/src/datatype.rs
index 207d14b8a3..2eae433f4d 100644
--- a/arrow-schema/src/datatype.rs
+++ b/arrow-schema/src/datatype.rs
@@ -313,13 +313,11 @@ pub enum DataType {
/// A single List array can store up to [`i32::MAX`] elements in total.
List(FieldRef),
- /// (NOT YET FULLY SUPPORTED) A list of some logical data type with
variable length.
+ /// A list of some logical data type with variable length.
///
/// Logically the same as [`List`], but the internal representation
differs in how child
/// data is referenced, allowing flexibility in how data is layed out.
///
- /// Note this data type is not yet fully supported. Using it with arrow
APIs may result in `panic`s.
- ///
/// [`List`]: Self::List
ListView(FieldRef),
/// A list of some logical data type with fixed length.
@@ -329,13 +327,11 @@ pub enum DataType {
/// A single LargeList array can store up to [`i64::MAX`] elements in
total.
LargeList(FieldRef),
- /// (NOT YET FULLY SUPPORTED) A list of some logical data type with
variable length and 64-bit offsets.
+ /// A list of some logical data type with variable length and 64-bit
offsets.
///
/// Logically the same as [`LargeList`], but the internal representation
differs in how child
/// data is referenced, allowing flexibility in how data is layed out.
///
- /// Note this data type is not yet fully supported. Using it with arrow
APIs may result in `panic`s.
- ///
/// [`LargeList`]: Self::LargeList
LargeListView(FieldRef),
/// A nested datatype that contains a number of sub-fields.
