feat: [Trace Stats] Move stats generation after trace obfuscation by lym953 · Pull Request #855 · DataDog/datadog-lambda-extension
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean by "have some data"? Do you mean traces should still be sent to Datadog even if stats fail to send?
Correct, before, WDYT?
What do you mean by "send the data back"?
As in, if you reply back to the tracer with a 400/500 status, will it re-send the tracer payload we failed to process?
I'm not sure how tracers work, but if we think this error is critical, the extension should surface it to the caller.
I agree, but in this case, it's not a processing error, it's more of a critical error on the extension, right? So we could say it's not the tracers fault, but ours(?