Results 781 comments of Pablo Baeyens

We could find inspiration on open-telemetry/opentelemetry-js/pull/5898

I am going to volunteer as a sponsor for this. The main reason behind this is that, while we no longer have a rule for accepting vendor exporters, I feel...

I think the same things I said on https://github.com/open-telemetry/opentelemetry-collector/issues/13171#issuecomment-2949537613 apply

We still need to update: - `exporterhelper.TimeoutSettings` - `NewDefaultTimeoutSettings` - `exporterhelper.QueueSettings` - `NewDefaultQueueSettings`

Taking a look at the output of `rg 'type \w*Setting' -tgo -i` it looks like this is indeed the case :rocket:

I will merge this next Wednesday unless objections are raised before then

I will merge this next Wednesday unless objections are raised before then

cc @andrzej-stencel, could this be a misconfiguration on Elastic's side?

`routingprocessor` needs a fix

@dmathieu This caused some conflicts on your other PRs, feel free to resolve conflicts on just one if you want to and ping me on it :)