Fix typesize in vlen index_codecs metadata

LDeakin · Jul 19, 2024 · e5ca43b · e5ca43b
1 parent 11c3e33
commit e5ca43b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/metadata/v3/codec/vlen.rs b/src/metadata/v3/codec/vlen.rs
@@ -77,7 +77,7 @@ mod tests {
     fn codec_vlen_compressed() {
         serde_json::from_str::<VlenCodecConfiguration>(r#"{
             "data_codecs": [{"name": "bytes"},{"name": "blosc","configuration": {"cname": "zstd", "clevel":5,"shuffle": "bitshuffle", "typesize":1,"blocksize":0}}],
-            "index_codecs": [{"name": "bytes","configuration": { "endian": "little" }},{"name": "blosc","configuration":{"cname": "zstd", "clevel":5,"shuffle": "shuffle", "typesize":2,"blocksize":0}}],
+            "index_codecs": [{"name": "bytes","configuration": { "endian": "little" }},{"name": "blosc","configuration":{"cname": "zstd", "clevel":5,"shuffle": "shuffle", "typesize":4,"blocksize":0}}],
             "index_data_type": "uint32"
         }"#).unwrap();
     }

diff --git a/tests/cities.rs b/tests/cities.rs
@@ -107,7 +107,7 @@ fn cities() -> Result<(), Box<dyn Error>> {
 
     let vlen_compressed_configuration: VlenCodecConfiguration = serde_json::from_str(r#"{
         "data_codecs": [{"name": "bytes"},{"name": "blosc","configuration": {"cname": "zstd", "clevel":5,"shuffle": "bitshuffle", "typesize":1,"blocksize":0}}],
-        "index_codecs": [{"name": "bytes","configuration": { "endian": "little" }},{"name": "blosc","configuration":{"cname": "zstd", "clevel":5,"shuffle": "shuffle", "typesize":2,"blocksize":0}}],
+        "index_codecs": [{"name": "bytes","configuration": { "endian": "little" }},{"name": "blosc","configuration":{"cname": "zstd", "clevel":5,"shuffle": "shuffle", "typesize":4,"blocksize":0}}],
         "index_data_type": "uint32"
     }"#)?;
     let vlen_compressed = Box::new(VlenCodec::new_with_configuration(&vlen_compressed_configuration)?);
@@ -126,7 +126,7 @@ fn cities() -> Result<(), Box<dyn Error>> {
     // | vlen_interleaved |             | 642196 |
     // | vlen_interleaved | zstd 5      | 362626 |
     // | vlen             |             | 642580 |
-    // | vlen             | zstd 5      | 358296 |
+    // | vlen             | zstd 5      | 346950 |
 
     Ok(())
 }