nit: I think, it would improve readability if proper argument naming would be used. e.g. `logicalTypeDecimal` or `ltDecimal` or similar. Moreover, `logicalTypeAnnotation` as method argument hides the local variable defined above.
[ Full content available at: https://github.com/apache/parquet-mr/pull/520 ] This message was relayed via gitbox.apache.org for [email protected]
