From aad55d58e4b6bc636686ab2f1c5c7261310cd51d Mon Sep 17 00:00:00 2001 From: Andrew Lamb Date: Tue, 17 Sep 2024 12:16:15 -0400 Subject: [PATCH] Remove "NOT YET FULLY SUPPORTED" comment from DataType::Utf8View/BinaryView (#6380) --- 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 1848c8b3f76e..b9cfc3d8a848 100644 --- a/arrow-schema/src/datatype.rs +++ b/arrow-schema/src/datatype.rs @@ -261,9 +261,7 @@ pub enum DataType { /// A single LargeBinary array can store up to [`i64::MAX`] bytes /// of binary data in total. LargeBinary, - /// (NOT YET FULLY SUPPORTED) Opaque binary data of variable length. - /// - /// Note this data type is not yet fully supported. Using it with arrow APIs may result in `panic`s. + /// Opaque binary data of variable length. /// /// Logically the same as [`Self::Binary`], but the internal representation uses a view /// struct that contains the string length and either the string's entire data @@ -280,9 +278,7 @@ pub enum DataType { /// A single LargeUtf8 array can store up to [`i64::MAX`] bytes /// of string data in total. LargeUtf8, - /// (NOT YET FULLY SUPPORTED) A variable-length string in Unicode with UTF-8 encoding - /// - /// Note this data type is not yet fully supported. Using it with arrow APIs may result in `panic`s. + /// A variable-length string in Unicode with UTF-8 encoding /// /// Logically the same as [`Self::Utf8`], but the internal representation uses a view /// struct that contains the string length and either the string's entire data