NotRecordRecordRecordAndSampled
Seems like we should pick between present tense or past tense and make it consistent. Either Recorded / RecordedAndSampled or Record / RecordAndSample.
NotRecord doesn't make sense. It's not correct English. Maybe just Skip or DontRecord. Personally I like just Skip better. It's already in an enum so it's always going to be qualified like SamplingDecision.Skip.
My vote:
SamplingDecision.SkipSamplingDecision.RecordSamplingDecision.RecordAndSample
https://github.com/open-telemetry/opentelemetry-dotnet/blob/master/src/OpenTelemetry/Trace/SamplingDecision.cs
NotRecordRecordRecordAndSampledSeems like we should pick between present tense or past tense and make it consistent. Either
Recorded/RecordedAndSampledorRecord/RecordAndSample.NotRecorddoesn't make sense. It's not correct English. Maybe justSkiporDontRecord. Personally I like justSkipbetter. It's already in an enum so it's always going to be qualified likeSamplingDecision.Skip.My vote:
SamplingDecision.SkipSamplingDecision.RecordSamplingDecision.RecordAndSamplehttps://github.com/open-telemetry/opentelemetry-dotnet/blob/master/src/OpenTelemetry/Trace/SamplingDecision.cs