From 0483b9c641ff9aa4e04dea4366d33ed65c330036 Mon Sep 17 00:00:00 2001 From: "Thibault \"bui\" Koechlin" Date: Fri, 20 May 2022 13:29:47 +0200 Subject: [PATCH] do not spew.Sdump() the invalid node on error. It leads to huge memory usage, especially if the parsers refers ie. datafile (#1550) --- pkg/parser/node.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/pkg/parser/node.go b/pkg/parser/node.go index 158c54d6c..dedafec80 100644 --- a/pkg/parser/node.go +++ b/pkg/parser/node.go @@ -545,12 +545,13 @@ func (n *Node) compile(pctx *UnixParserCtx, ectx EnricherCtx) error { if !valid { /* node is empty, error force return */ - n.Logger.Infof("Node is empty: %s", spew.Sdump(n)) + n.Logger.Error("Node is empty or invalid, abort") n.Stage = "" + return fmt.Errorf("Node is empty") } + if err := n.validate(pctx, ectx); err != nil { return err - //n.logger.Fatalf("Node is invalid : %s", err) } return nil }