Fix incorrect attributes budget return

blt · blt · commit 3aa693a97fd8 · 2025-07-22T12:03:41.000-07:00
Signed-off-by: Brian L. Troutwine &lt;brian.troutwine@datadoghq.com&gt;
diff --git a/lading_payload/src/opentelemetry/log/templates.rs b/lading_payload/src/opentelemetry/log/templates.rs
@@ -94,6 +94,13 @@ impl<'a> crate::SizedGenerator<'a> for LogTemplateGenerator {
                 if inner_budget == original_budget {
                     return Err(GeneratorError::SizeExhausted);
                 }
+                // If attribute population was partial -- signaled by
+                // inner_budget being drawn down on -- we return the budget back
+                // and live with no attributes. We do this because OTLP is
+                // expensive to generate and we may be able to create a
+                // reasonable instance of a `LogRecord` with the budget
+                // otherwise on hand.
+                inner_budget = original_budget;
                 Vec::new()
             }
             Err(e) => return Err(e),

Original file line number	Diff line number	Diff line change
`@@ -94,6 +94,13 @@ impl<'a> crate::SizedGenerator<'a> for LogTemplateGenerator {`
`94`	`94`	`if inner_budget == original_budget {`
`95`	`95`	`return Err(GeneratorError::SizeExhausted);`
`96`	`96`	`}`
	`97`	`+ // If attribute population was partial -- signaled by`
	`98`	`+ // inner_budget being drawn down on -- we return the budget back`
	`99`	`+ // and live with no attributes. We do this because OTLP is`
	`100`	`+ // expensive to generate and we may be able to create a`
	`101`	+ // reasonable instance of a `LogRecord` with the budget
	`102`	`+ // otherwise on hand.`
	`103`	`+ inner_budget = original_budget;`
`97`	`104`	`Vec::new()`
`98`	`105`	`}`
`99`	`106`	`Err(e) => return Err(e),`