Skip to content

Commit 72f2396

Browse files
mhk197joseph-isaacs
authored andcommitted
fix comment
Signed-off-by: Matt Katz <mhkatz97@gmail.com>
1 parent b8dbf6a commit 72f2396

1 file changed

Lines changed: 1 addition & 12 deletions

File tree

‎vortex-test/compat-gen/src/fixtures/arrays/synthetic/encodings/decimal_byte_parts_v2.rs‎

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,7 @@
11
// SPDX-License-Identifier: Apache-2.0
22
// SPDX-FileCopyrightText: Copyright the Vortex contributors
33

4-
//! Wide `DecimalByteParts` fixtures: values that need lower parts.
5-
//!
6-
//! These live in their own fixture file rather than as extra columns on
7-
//! `decimal_byte_parts.vortex` because a fixture's `build()` is immutable once published.
8-
//! `check` compares files written by older releases against what `build()` produces today,
9-
//! so changing an existing fixture's schema fails the check against every previously
10-
//! published version — see "Fixture evolution" in `DESIGN.md`, which requires a new fixture
11-
//! file with a new name for a new type, encoding, or structural pattern.
12-
//!
13-
//! So `decimal_byte_parts.vortex` keeps testing exactly what it always did, decimals whose
14-
//! values fit a single signed part, and the MSP-plus-lower-parts layout added alongside it
15-
//! is covered here instead.
4+
//! `DecimalByteParts` fixture for wide decimal values that need lower parts.
165
176
use vortex::array::ArrayId;
187
use vortex::array::ArrayRef;

0 commit comments

Comments
 (0)