zeroshade commented on a change in pull request #11570:
URL: https://github.com/apache/arrow/pull/11570#discussion_r738807666
##########
File path: go/parquet/writer_properties.go
##########
@@ -18,7 +18,7 @@ package parquet
import (
"github.com/apache/arrow/go/arrow/memory"
Review comment:
I can't upgrade the dependency to v6 in this branch until there exists a
`go/arrow/v6.X.Y` tag which has the go.mod containing the `/v6` in the module
name. The `go.sum` file in here is a checksum of the dependencies, and I can't
get it to upgrade to the v6 until the tag exists with a correct go.mod.
The only way to have the `/v6` suffix here would be for us to merge the
arrow and parquet packages into a single module with the two separate packages
instead of maintaining them as two entirely separate modules. Which i'm
increasingly thinking we should do.
--
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]
