From bfca339da499423e32fdfdd47e6efe2a8f293b9a Mon Sep 17 00:00:00 2001 From: Adrian Garcia Badaracco <1755071+adriangb@users.noreply.github.com> Date: Thu, 4 Jun 2026 13:04:50 -0400 Subject: [PATCH] test(parquet): drop confusing `main` reference in page-roundtrip test comment Follow-up to #9972. The test comment compared granular-mode page output to `main`, which is confusing now that the PR has merged into main. Compare to the default batched path instead. Co-Authored-By: Claude Opus 4.8 (1M context) --- parquet/src/arrow/arrow_writer/mod.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/parquet/src/arrow/arrow_writer/mod.rs b/parquet/src/arrow/arrow_writer/mod.rs index 86e6cf081fed..873ffb826eca 100644 --- a/parquet/src/arrow/arrow_writer/mod.rs +++ b/parquet/src/arrow/arrow_writer/mod.rs @@ -4906,10 +4906,10 @@ mod tests { #[test] fn test_arrow_writer_granular_mode_roundtrip() { - // Granular mode subdivides chunks and writes more pages than - // `main`. Make sure the data we write back is bit-identical to - // what went in — page-count assertions elsewhere only prove - // pages were cut, not that the encoded data is correct. + // Granular mode subdivides chunks and writes more pages than the + // default batched path. Make sure the data we write back is + // bit-identical to what went in — page-count assertions elsewhere + // only prove pages were cut, not that the encoded data is correct. // // Mix value sizes so that the cumulative-byte-budget cutoff // lands mid-chunk, exercising both batched and granular paths