Steven Bouwkamp
Steven Bouwkamp
> One _slight_ concern: you "only" touched 70 files here, but we have nearly 1700 snapshots - how confident are you that those snapshots definitely don't _need_ updating vs just...
> (I guess this is related to #4017 ?) Yes it is @ivoanjo! I posted in the slack channel some context surrounding this I didn't link to it yet as...
> Hey @bouwkast thoughts on merging this in? Looks like a nice improvement I was out on PTO, looked like I just had a couple of conversations that needed to...
> Any updates on the progress of this PR? This would be really useful for us! Hi @johan-lindqvist, work has stalled somewhat on this, but I'll carve out some time...
I _think_ this has been an issue for a while 😬
For reference: - https://github.com/DataDog/dd-trace-dotnet/blob/1ddd57628073106ec7da3b3828e1ff6dd5d4c748/tracer/src/Datadog.Trace/Configuration/Schema/ClientSchema.cs#L18 - https://github.com/DataDog/dd-trace-dotnet/blob/1ddd57628073106ec7da3b3828e1ff6dd5d4c748/tracer/src/Datadog.Trace/Configuration/Schema/DatabaseSchema.cs#L20
> Isn't it the other way around, that enabling schema v1 enabled several config knobs, including `RemoveClientServiceNamesEnabled`? (but they can also be enabled individually?) cc @zacharycmontoya This is something that...
No longer relevant
Closing can always revisit later if needed
Closing as we already have in place several different strategies to do this