sbinet commented on a change in pull request #9671:
URL: https://github.com/apache/arrow/pull/9671#discussion_r596219612
##########
File path: go/parquet/internal/bmi/_lib/bitmap_bmi2.s
##########
@@ -0,0 +1,174 @@
+ .text
Review comment:
well, removing those would require people to have all the tooling
installed to regenerate them before being able to use go-parquet, right?
I'd rather keep all the files checked in that are needed to preserve the
usual seamless "go get" installation workflow.
----------------------------------------------------------------
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]
