(arrow-rs) branch main updated: Move arrow_writer_layout.rs into arrow_writer/layout.rs (#10542)

[alamb]

This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git


The following commit(s) were added to refs/heads/main by this push:
     new 7e4432e997 Move arrow_writer_layout.rs into arrow_writer/layout.rs 
(#10542)
7e4432e997 is described below

commit 7e4432e997e5b0be31a5641e078a889d82b6f61b
Author: Andrew Lamb <and...@nerdnetworks.org>
AuthorDate: Wed Aug 5 10:56:02 2026 -0400

    Move arrow_writer_layout.rs into arrow_writer/layout.rs (#10542)
    
    # Which issue does this PR close?
    
    - Part of #10540.
    
    # Rationale for this change
    
    Consolidate the scattered parquet end-to-end tests: the `ArrowWriter`
    page layout tests belong with the other `arrow_writer` integration tests
    rather than in a separate test binary.
    
    # What changes are included in this PR?
    
    - Move `parquet/tests/arrow_writer.rs` to
    `parquet/tests/arrow_writer/mod.rs`
    - Move `parquet/tests/arrow_writer_layout.rs` to
    `parquet/tests/arrow_writer/layout.rs` (unchanged, now a submodule)
    - Remove the separate `arrow_writer_layout` test target from
    `Cargo.toml`
    
    # Are these changes tested?
    
    Yes, this PR only moves existing tests. All 14 tests pass via `cargo
    test -p parquet --features arrow --test arrow_writer`.
    
    # Are there any user-facing changes?
    
    No, test-only change.
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    Co-authored-by: Claude Fable 5 <nore...@anthropic.com>
---
 parquet/Cargo.toml                                               | 6 +-----
 parquet/tests/{arrow_writer_layout.rs => arrow_writer/layout.rs} | 0
 parquet/tests/{arrow_writer.rs => arrow_writer/mod.rs}           | 2 ++
 3 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/parquet/Cargo.toml b/parquet/Cargo.toml
index 4e209b52ec..662bb2b422 100644
--- a/parquet/Cargo.toml
+++ b/parquet/Cargo.toml
@@ -164,10 +164,6 @@ name = "object_store"
 required-features = ["arrow", "async"]
 path = "./examples/object_store.rs"
 
-[[test]]
-name = "arrow_writer_layout"
-required-features = ["arrow"]
-
 [[test]]
 name = "arrow_reader"
 required-features = ["arrow"]
@@ -176,7 +172,7 @@ path = "./tests/arrow_reader/mod.rs"
 [[test]]
 name = "arrow_writer"
 required-features = ["arrow"]
-path = "./tests/arrow_writer.rs"
+path = "./tests/arrow_writer/mod.rs"
 
 [[test]]
 name = "ieee754_nan_interop"
diff --git a/parquet/tests/arrow_writer_layout.rs 
b/parquet/tests/arrow_writer/layout.rs
similarity index 100%
rename from parquet/tests/arrow_writer_layout.rs
rename to parquet/tests/arrow_writer/layout.rs
diff --git a/parquet/tests/arrow_writer.rs b/parquet/tests/arrow_writer/mod.rs
similarity index 99%
rename from parquet/tests/arrow_writer.rs
rename to parquet/tests/arrow_writer/mod.rs
index ce65ba01fe..b0bf3bcb5a 100644
--- a/parquet/tests/arrow_writer.rs
+++ b/parquet/tests/arrow_writer/mod.rs
@@ -17,6 +17,8 @@
 
 //! Tests for [`ArrowWriter`]
 
+mod layout;
+
 use std::alloc::{GlobalAlloc, Layout, System};
 use std::cell::Cell;
 use std::fs::File;