[chore] Avoid long switch call in zap.Any if the logs will not be logged (#39917)

This is a very large performance impact for transform heavy services.

Signed-off-by: Bogdan Drutu <[email protected]>

Author: bogdandrutu

pkg/ottl/context_inferrer.go

@@ -109,12 +109,14 @@ func (s *priorityContextInferrer) infer(statements []string, conditions []string
 			return "", err
 		}
 	}
-	s.telemetrySettings.Logger.Debug("Inferring context from statements and conditions",
-		zap.Strings("candidates", maps.Keys(s.contextCandidate)),
-		zap.Any("priority", s.contextPriority),
-		zap.Strings("statements", statements),
-		zap.Strings("conditions", conditions),
-	)
+	if s.telemetrySettings.Logger.Core().Enabled(zap.DebugLevel) {
+		s.telemetrySettings.Logger.Debug("Inferring context from statements and conditions",
+			zap.Strings("candidates", maps.Keys(s.contextCandidate)),
+			zap.Any("priority", s.contextPriority),
+			zap.Strings("statements", statements),
+			zap.Strings("conditions", conditions),
+		)
+	}
 	return s.inferFromHints(append(statementsHints, conditionsHints...))
 }
 

pkg/ottl/parser.go

@@ -32,7 +32,7 @@ type Statement[K any] struct {
 func (s *Statement[K]) Execute(ctx context.Context, tCtx K) (any, bool, error) {
 	condition, err := s.condition.Eval(ctx, tCtx)
 	defer func() {
-		if s.telemetrySettings.Logger != nil {
+		if s.telemetrySettings.Logger.Core().Enabled(zap.DebugLevel) {
 			s.telemetrySettings.Logger.Debug("TransformContext after statement execution", zap.String("statement", s.origText), zap.Bool("condition matched", condition), zap.Any("TransformContext", tCtx))
 		}
 	}()
@@ -381,7 +381,9 @@ func NewStatementSequence[K any](statements []*Statement[K], telemetrySettings c
 // When the ErrorMode of the StatementSequence is `ignore`, errors are logged and execution continues to the next statement.
 // When the ErrorMode of the StatementSequence is `silent`, errors are not logged and execution continues to the next statement.
 func (s *StatementSequence[K]) Execute(ctx context.Context, tCtx K) error {
-	s.telemetrySettings.Logger.Debug("initial TransformContext before executing StatementSequence", zap.Any("TransformContext", tCtx))
+	if s.telemetrySettings.Logger.Core().Enabled(zap.DebugLevel) {
+		s.telemetrySettings.Logger.Debug("initial TransformContext before executing StatementSequence", zap.Any("TransformContext", tCtx))
+	}
 	for _, statement := range s.statements {
 		_, _, err := statement.Execute(ctx, tCtx)
 		if err != nil {
@@ -454,7 +456,9 @@ func (c *ConditionSequence[K]) Eval(ctx context.Context, tCtx K) (bool, error) {
 	var atLeastOneMatch bool
 	for _, condition := range c.conditions {
 		match, err := condition.Eval(ctx, tCtx)
-		c.telemetrySettings.Logger.Debug("condition evaluation result", zap.String("condition", condition.origText), zap.Bool("match", match), zap.Any("TransformContext", tCtx))
+		if c.telemetrySettings.Logger.Core().Enabled(zap.DebugLevel) {
+			c.telemetrySettings.Logger.Debug("condition evaluation result", zap.String("condition", condition.origText), zap.Bool("match", match), zap.Any("TransformContext", tCtx))
+		}
 		if err != nil {
 			if c.errorMode == PropagateError {
 				err = fmt.Errorf("failed to eval condition: %v, %w", condition.origText, err)