From f630d17f0c79dfdf8453616207537fc57d6365af Mon Sep 17 00:00:00 2001
From: Ryan <ryan@ryanj.net>
Date: Sun, 28 Aug 2022 23:12:57 +1000
Subject: [PATCH] Mention parquet compatibility under `TimeUnit`.

`TimeUnit::Second` will not store a logical type when converted to parquet format.
---
 src/datatypes/mod.rs | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/datatypes/mod.rs b/src/datatypes/mod.rs
index 4be369da0a2..658ca67aef9 100644
--- a/src/datatypes/mod.rs
+++ b/src/datatypes/mod.rs
@@ -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 {