C-Loftus commented on code in PR #1028:
URL: https://github.com/apache/arrow-go/pull/1028#discussion_r3658316362
##########
parquet/pqarrow/schema.go:
##########
@@ -121,24 +121,24 @@ func (sm *SchemaManifest) GetFieldIndices(indices []int)
([]int, error) {
}
// ExtensionCustomParquetType is an interface that Arrow ExtensionTypes may
implement
-// to specify the target LogicalType to use when converting to Parquet.
+// to specify the target LogicalType to use when converting to Parquet on
write.
Review Comment:
Changed to `on write` since it makes it clearer that it is one side of the
read/write path for arrow <-> parquet but worth aligning if there are any cases
in which this would be used beyond writing
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
