open-telemetry / otel-arrow

Protocol and libraries for sending and receiving OpenTelemetry data using Apache Arrow
Apache License 2.0
68 stars 13 forks source link

Fix inFlightWG race in the OTel-Arrow receiver #212

Closed jmacd closed 3 months ago

jmacd commented 3 months ago

Part of https://github.com/open-telemetry/otel-arrow/pull/210.

Mainly, fixes a race in the use of inFlightWG which is meant to be per-stream but is per-Receiver presently.

Secondly, re-order and rename of the fields passed to the "arrow stream error" log statement, so that it matches the Exporter. This is used as the basis of a test for logging consistency and there was otherwise an unintentional disagreement ("which" and "where").