Skip to content

Commit

Permalink
Encoding ID
Browse files Browse the repository at this point in the history
  • Loading branch information
gatesn committed Mar 22, 2024
1 parent 5b922b0 commit 59c554e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
2 changes: 1 addition & 1 deletion vortex-alp/src/array.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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())),
}
}

Expand Down
7 changes: 1 addition & 6 deletions vortex-fastlanes/src/for/compress.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -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))
Expand Down

0 comments on commit 59c554e

Please sign in to comment.