tanmayrauth commented on issue #1236:
URL: https://github.com/apache/iceberg-go/issues/1236#issuecomment-4753625090
Thanks @laskoviymishka! Ack on all points:
- OTel surface → track Java (#16250), marked experimental until it merges,
cross-linked as a dependency on PR 7.
- Interop tests against real Java-produced JSON as a hard requirement (PR
2) — covering both the REST envelope and the commit-report ↔ snapshot-summary
name mapping.
- AlwaysTrue documented as a lossy v1 stopgap; real filter serialization
will match ExpressionParser.toJson.
For the remaining open questions I'll go with the proposed defaults
(registry + programmatic selection, in-module metrics/otel), and lean toward a
plain internal atomic.Int64 struct for accumulation — I'll build it that way in
PR 1 so it's concrete to review.
Starting on PRs 1–6 now. 🙏
--
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]
