Remove unused StructOfChunks in favour of VortexLayoutStrategy (#2038)

Author: robert3005

vortex-file/src/lib.rs

@@ -98,7 +98,6 @@ pub mod io;
 mod open;
 pub mod read;
 pub mod segments;
-mod strategy;
 #[cfg(test)]
 mod tests;
 mod writer;

vortex-file/src/writer.rs

@@ -3,11 +3,10 @@ use vortex_array::stream::ArrayStream;
 use vortex_error::{vortex_bail, vortex_err, VortexExpect, VortexResult};
 use vortex_flatbuffers::{FlatBuffer, FlatBufferRoot, WriteFlatBuffer, WriteFlatBufferExt};
 use vortex_io::VortexWrite;
-use vortex_layout::strategies::LayoutStrategy;
+use vortex_layout::strategies::{LayoutStrategy, VortexLayoutStrategy};
 
 use crate::footer::{FileLayout, Postscript, Segment};
 use crate::segments::writer::BufferedSegmentWriter;
-use crate::strategy::VortexLayoutStrategy;
 use crate::{EOF_SIZE, MAGIC_BYTES, MAX_FOOTER_SIZE, VERSION};
 
 pub struct VortexWriteOptions {

vortex-layout/src/strategies/mod.rs

@@ -6,9 +6,9 @@
 //! otherwise manipulate the chunks of data enabling experimentation with different strategies
 //! all while remaining independent of the read code.
 
-mod struct_of_chunks;
+mod strategy;
 
-pub use struct_of_chunks::*;
+pub use strategy::*;
 use vortex_array::ArrayData;
 use vortex_dtype::DType;
 use vortex_error::VortexResult;

vortex-file/src/strategy.rs vortex-layout/src/strategies/strategy.rsvortex-file/src/strategy.rs renamed to vortex-layout/src/strategies/strategy.rs

@@ -2,9 +2,10 @@
 
 use vortex_dtype::DType;
 use vortex_error::VortexResult;
-use vortex_layout::layouts::chunked::writer::{ChunkedLayoutOptions, ChunkedLayoutWriter};
-use vortex_layout::layouts::struct_::writer::StructLayoutWriter;
-use vortex_layout::strategies::{LayoutStrategy, LayoutWriter, LayoutWriterExt};
+
+use crate::layouts::chunked::writer::{ChunkedLayoutOptions, ChunkedLayoutWriter};
+use crate::layouts::struct_::writer::StructLayoutWriter;
+use crate::strategies::{LayoutStrategy, LayoutWriter, LayoutWriterExt};
 
 /// The default Vortex file layout strategy.
 ///