alamb commented on a change in pull request #9731:
URL: https://github.com/apache/arrow/pull/9731#discussion_r596737499
##########
File path: rust/parquet/src/basic.rs
##########
@@ -156,6 +158,12 @@ pub enum ConvertedType {
// Mirrors `parquet::LogicalType`
/// Logical types used by version 2 of the Parquet format.
+///
+/// *Upgrade Note*: This is an *entirely new* struct as of version
Review comment:
I think it is important to put a hint as close to where people will hit
the compiler error as possible (e.g. I wouldn't have known to look in the
README when I hit the error on upgrade).
That being said, I think you are right that the actual content doesn't need
to be duplicated here -- I will change it to point people at the readme for
further details.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
