Skip to content
Closed
Show file tree
Hide file tree
Changes from 8 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions arrow-schema/src/error.rs
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ pub enum ArrowError {
DictionaryKeyOverflowError,
/// Error when the run end index in a REE array is bigger than the array length
RunEndIndexOverflowError,
/// Error during Variant operations in `arrow-variant`.
VariantError(String),
}

impl ArrowError {
Expand Down Expand Up @@ -128,9 +126,6 @@ impl Display for ArrowError {
ArrowError::RunEndIndexOverflowError => {
write!(f, "Run end encoded array index overflow error")
}
ArrowError::VariantError(desc) => {
write!(f, "Variant error: {desc}")
}
}
}
}
Expand Down
12 changes: 0 additions & 12 deletions arrow-schema/src/extension/canonical/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,6 @@ mod uuid;
pub use uuid::Uuid;
mod variable_shape_tensor;
pub use variable_shape_tensor::{VariableShapeTensor, VariableShapeTensorMetadata};
mod variant;
pub use variant::Variant;

use crate::{ArrowError, Field};

Expand Down Expand Up @@ -79,9 +77,6 @@ pub enum CanonicalExtensionType {
///
/// <https://arrow.apache.org/docs/format/CanonicalExtensions.html#bit-boolean>
Bool8(Bool8),

/// The extension type for `Variant`.
Variant(Variant),
}

impl TryFrom<&Field> for CanonicalExtensionType {
Expand All @@ -98,7 +93,6 @@ impl TryFrom<&Field> for CanonicalExtensionType {
Uuid::NAME => value.try_extension_type::<Uuid>().map(Into::into),
Opaque::NAME => value.try_extension_type::<Opaque>().map(Into::into),
Bool8::NAME => value.try_extension_type::<Bool8>().map(Into::into),
Variant::NAME => value.try_extension_type::<Variant>().map(Into::into),
_ => Err(ArrowError::InvalidArgumentError(format!("Unsupported canonical extension type: {name}"))),
},
// Name missing the expected prefix
Expand Down Expand Up @@ -146,9 +140,3 @@ impl From<Bool8> for CanonicalExtensionType {
CanonicalExtensionType::Bool8(value)
}
}

impl From<Variant> for CanonicalExtensionType {
fn from(value: Variant) -> Self {
CanonicalExtensionType::Variant(value)
}
}
286 changes: 0 additions & 286 deletions arrow-schema/src/extension/canonical/variant.rs

This file was deleted.

2 changes: 1 addition & 1 deletion arrow-variant/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
[package]
name = "arrow-variant"
version = { workspace = true }
description = "JSON to Arrow Variant conversion utilities"
description = "Rust API for reading/writing Apache Parquet Variant values"
homepage = { workspace = true }
repository = { workspace = true }
authors = { workspace = true }
Expand Down
Loading
Loading