Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Commit

Permalink
Mention parquet compatibility under TimeUnit.
Browse files Browse the repository at this point in the history
`TimeUnit::Second` will not store a logical type when converted to parquet format.
  • Loading branch information
Ryan committed Aug 28, 2022
1 parent c6f1427 commit f630d17
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/datatypes/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -191,6 +191,11 @@ impl UnionMode {
}

/// The time units defined in Arrow.
///
/// # Parquet Compatibility
/// As of [parquet-format v2.9](https://github.com/apache/parquet-format/blob/master/LogicalTypes.md)
/// `TimeUnit::Second`' is not a defined parquet logical type. The use of `TimeUnit::Second` will
/// result in no logical type being stored within a parquet file.
#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
#[cfg_attr(feature = "serde_types", derive(Serialize, Deserialize))]
pub enum TimeUnit {
Expand Down

0 comments on commit f630d17

Please sign in to comment.