Przeglądaj źródła

do not spew.Sdump() the invalid node on error. It leads to huge memory usage, especially if the parsers refers ie. datafile (#1550)

Thibault "bui" Koechlin 3 lat temu
rodzic
commit
0483b9c641
1 zmienionych plików z 3 dodań i 2 usunięć
  1. 3 2
      pkg/parser/node.go

+ 3 - 2
pkg/parser/node.go

@@ -545,12 +545,13 @@ func (n *Node) compile(pctx *UnixParserCtx, ectx EnricherCtx) error {
 
 
 	if !valid {
 	if !valid {
 		/* node is empty, error force return */
 		/* 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 = ""
 		n.Stage = ""
+		return fmt.Errorf("Node is empty")
 	}
 	}
+
 	if err := n.validate(pctx, ectx); err != nil {
 	if err := n.validate(pctx, ectx); err != nil {
 		return err
 		return err
-		//n.logger.Fatalf("Node is invalid : %s", err)
 	}
 	}
 	return nil
 	return nil
 }
 }