From 59c554e18d8240fe6679cd7a9ae0422d2638be71 Mon Sep 17 00:00:00 2001
From: Nicholas Gates <nick@nickgates.com>
Date: Fri, 22 Mar 2024 13:19:28 +0000
Subject: [PATCH] Encoding ID

---
 vortex-alp/src/array.rs              | 2 +-
 vortex-fastlanes/src/for/compress.rs | 7 +------
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/vortex-alp/src/array.rs b/vortex-alp/src/array.rs
index 3e457aa616..c3233fb10a 100644
--- a/vortex-alp/src/array.rs
+++ b/vortex-alp/src/array.rs
@@ -51,7 +51,7 @@ impl ALPArray {
     pub fn encode(array: &dyn Array) -> VortexResult<ArrayRef> {
         match ArrayKind::from(array) {
             ArrayKind::Primitive(p) => Ok(alp_encode(p)?.into_array()),
-            _ => Err(VortexError::InvalidEncoding(array.encoding().id())),
+            _ => Err(VortexError::InvalidEncoding(array.encoding().id().clone())),
         }
     }
 
diff --git a/vortex-fastlanes/src/for/compress.rs b/vortex-fastlanes/src/for/compress.rs
index 68222e88b5..b030e443d9 100644
--- a/vortex-fastlanes/src/for/compress.rs
+++ b/vortex-fastlanes/src/for/compress.rs
@@ -139,7 +139,6 @@ mod test {
     use std::collections::HashSet;
     use std::sync::Arc;
 
-    use vortex::array::primitive::PrimitiveEncoding;
     use vortex::array::Encoding;
 
     use crate::BitPackedEncoding;
@@ -149,11 +148,7 @@ mod test {
     fn compress_ctx() -> CompressCtx {
         let cfg = CompressConfig::new(
             // We need some BitPacking else we will need choose FoR.
-            HashSet::from([
-                PrimitiveEncoding.id(),
-                FoREncoding.id(),
-                BitPackedEncoding.id(),
-            ]),
+            HashSet::from([FoREncoding.id(), BitPackedEncoding.id()]),
             HashSet::default(),
         );
         CompressCtx::new(Arc::new(cfg))