zeroshade commented on PR #1537:
URL: https://github.com/apache/iceberg-go/pull/1537#issuecomment-5096887798
@hankivstmb heads-up — main is currently failing to compile from a merge
collision between this PR and #1426. This PR added the empty-batch sentinel at
table/arrow_scanner.go:1027 using internal.Enumerated[...], while #1426 (merged
around the same time) renamed that file's table/internal import to the alias
tblutils. Neither diff conflicted textually, but the merged file now has
'undefined: internal', which fails the Go and Audit-and-Verify workflows on
main (cascading to codec via the table import). One-line fix at
arrow_scanner.go:1027: change internal.Enumerated[arrow.RecordBatch]{ to
tblutils.Enumerated[arrow.RecordBatch]{ . Could you open a quick fix PR?
Merge-ordering on our side, sorry about that — happy to fast-track the review.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
