Alan West

Results 86 comments of Alan West

> My main goal here is to fix a critical bug: The current code allows for infinite retries. I would support an option to configure the max retry attempts if...

I agree that a configurable max number of retries would be useful, but I'd like to see the specification permit this configuration first. I opened an issue against the specification...

With the exponential backoff I don't think you achieve your goal of a "fail-fast". How long it will take a fixed number of retries is indeterministic. At this time, I...

Yes there are definitely going to be new cases we'll need to support for other database systems. Are we at least good with SQL server? My original implementation was ok...

Added a simple test illustrating status quo for MSSQL bracketed identifiers https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/3616. Also shared my thoughts on these edge cases. @stevejgordon if my PR is redundant based on the work...

> Allow instrumentations to report process.cpu.time for their own process (resource attributes would differentiate it from process.cpu.time reported by the OpenTelemetry Collector) Did you also mean `process.cpu.utilization`? Looks this issue...