ArnavBalyan commented on code in PR #704:
URL: https://github.com/apache/arrow-go/pull/704#discussion_r3614932407
##########
parquet/file/column_writer_test.go:
##########
@@ -502,6 +502,15 @@ func (p *PrimitiveWriterTestSuite)
TestRequiredByteStreamSplit() {
}
}
+func (p *PrimitiveWriterTestSuite) TestRequiredAlp() {
Review Comment:
Hey Prateek, thanks for taking a look. The code was tested with cross compat
tests where any external file can be provided that the go implementation can
decode. We may have to do this for arrow-go. I should have bandwidth to take
this fwd early August.
Please feel free to use this change if you have time before thanks!
--
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]
