Minor simplification of the serde dependency

Cargo.toml

@@ -172,7 +172,8 @@ roaring = "0.11.0"
 rstest = "0.26"
 rstest_reuse = "0.7.0"
 rustc-hash = "2.1"
-serde = "1.0.217"
+serde = "1.0.220"
+serde_core = "1.0.220"
 serde_json = "1.0.116"
 serde_test = "1.0.176"
 simdutf8 = "0.1.5"

vortex-array/Cargo.toml

@@ -54,7 +54,6 @@ rand = { workspace = true }
 rstest = { workspace = true, optional = true }
 rstest_reuse = { workspace = true, optional = true }
 rustc-hash = { workspace = true }
-serde = { workspace = true, features = ["derive"] }
 simdutf8 = { workspace = true }
 static_assertions = { workspace = true }
 tabled = { workspace = true, optional = true, default-features = false, features = [

vortex-array/src/patches.rs

@@ -8,7 +8,6 @@ use std::ops::Range;
 
 use arrow_buffer::BooleanBuffer;
 use itertools::Itertools;
-use serde::{Deserialize, Serialize};
 use vortex_buffer::BufferMut;
 use vortex_dtype::Nullability::NonNullable;
 use vortex_dtype::{
@@ -26,7 +25,7 @@ use crate::search_sorted::{SearchResult, SearchSorted, SearchSortedSide};
 use crate::vtable::ValidityHelper;
 use crate::{Array, ArrayRef, IntoArray, ToCanonical};
 
-#[derive(Copy, Clone, Serialize, Deserialize, prost::Message)]
+#[derive(Copy, Clone, prost::Message)]
 pub struct PatchesMetadata {
     #[prost(uint64, tag = "1")]
     len: u64,

vortex-buffer/Cargo.toml

@@ -20,7 +20,7 @@ all-features = true
 arrow = ["dep:arrow-buffer"]
 cuda = ["dep:cudarc"]
 memmap2 = ["dep:memmap2"]
-serde = ["dep:serde"]
+serde = ["dep:serde_core"]
 warn-copy = ["dep:log"]
 
 [dependencies]
@@ -31,7 +31,7 @@ itertools = { workspace = true }
 log = { workspace = true, optional = true }
 memmap2 = { workspace = true, optional = true }
 num-traits = { workspace = true }
-serde = { workspace = true, optional = true }
+serde_core = { workspace = true, optional = true }
 simdutf8 = { workspace = true }
 vortex-error = { workspace = true }
 

vortex-buffer/src/serde.rs

@@ -3,8 +3,8 @@
 
 use std::marker::PhantomData;
 
-use serde::de::Visitor;
-use serde::{Deserialize, Deserializer, Serialize, Serializer};
+use serde_core::de::Visitor;
+use serde_core::{Deserialize, Deserializer, Serialize, Serializer};
 
 use crate::{Alignment, Buffer, BufferMut, ByteBuffer};
 
@@ -44,31 +44,31 @@ where
 
     fn visit_byte_buf<E>(self, v: Vec<u8>) -> Result<Self::Value, E>
     where
-        E: serde::de::Error,
+        E: serde_core::de::Error,
     {
         let bytes = ByteBuffer::copy_from_aligned(v, Alignment::of::<T>());
         Ok(Buffer::from_byte_buffer(bytes))
     }
 
     fn visit_borrowed_bytes<E>(self, v: &'de [u8]) -> Result<Self::Value, E>
     where
-        E: serde::de::Error,
+        E: serde_core::de::Error,
     {
         let bytes = ByteBuffer::copy_from_aligned(v, Alignment::of::<T>());
         Ok(Buffer::from_byte_buffer(bytes))
     }
 
     fn visit_bytes<E>(self, v: &[u8]) -> Result<Self::Value, E>
     where
-        E: serde::de::Error,
+        E: serde_core::de::Error,
     {
         let bytes = ByteBuffer::copy_from_aligned(v, Alignment::of::<T>());
         Ok(Buffer::from_byte_buffer(bytes))
     }
 
     fn visit_seq<A>(self, mut seq: A) -> Result<Self::Value, A::Error>
     where
-        A: serde::de::SeqAccess<'de>,
+        A: serde_core::de::SeqAccess<'de>,
     {
         let mut buffer = seq
             .size_hint()

vortex-dtype/src/serde/mod.rs

@@ -6,6 +6,7 @@ pub mod flatbuffers;
 mod proto;
 
 #[allow(clippy::module_inception)]
+#[cfg(feature = "serde")]
 mod serde;
 
 #[cfg(test)]

vortex-dtype/src/serde/serde.rs

@@ -1,8 +1,6 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
-#![cfg(feature = "serde")]
-
 use serde::{Deserialize, Deserializer, Serialize, Serializer};
 
 use crate::Nullability;