Skip to content

Commit 6aa9c47

Browse files
committed
ai review
1 parent a13803c commit 6aa9c47

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

kernel/src/lib.rs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -793,10 +793,13 @@ pub trait ParquetHandler: AsAny {
793793
/// # Type coercion
794794
///
795795
/// When `physical_schema` requests a type different from the matched Parquet column, the reader
796-
/// must perform every conversion allowed by the [Delta protocol type widening rules]. Examples
797-
/// include `INTEGER` -> `LONG`, `FLOAT` -> `DOUBLE`, integer types -> `DECIMAL`, `DECIMAL`
798-
/// precision or scale increases, and `DATE` -> `TIMESTAMP_NTZ`. See the protocol for the
799-
/// complete list.
796+
/// must coerce it to the requested type when the conversion is allowed by the [Delta protocol
797+
/// type widening rules]. Examples include `INTEGER` -> `LONG`, `FLOAT` -> `DOUBLE`, and `DATE`
798+
/// -> `TIMESTAMP_NTZ`. See the protocol for the complete list.
799+
///
800+
/// Kernel additionally requires readers to support `INT32` -> `DATE` and `INT64` ->
801+
/// `TIMESTAMP` / `TIMESTAMP_NTZ` conversions for checkpoint `stats_parsed` and
802+
/// `partitionValues_parsed` fields.
800803
///
801804
/// Timestamp columns MUST be normalized to the protocol specified microsecond precision: a
802805
/// `TIMESTAMP(MILLIS)` (or any other non-microsecond unit) column read into a `TIMESTAMP` /

0 commit comments

Comments
 (0)