tanmayrauth commented on issue #1091:
URL: https://github.com/apache/iceberg-go/issues/1091#issuecomment-4491682602
I'd suggest including both in the same PR, the fix is identical in spirit
(missing metric for an optional float field should mean "unknown, not zero"),
the code lives in the same file, and splitting it would just create a second
PR with the same test structure and review overhead. It's a localized, cohesive
change.
--
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]
