logsql/stats: clarify error message for unsafe pipes before stats (#878)

func25 · web-flow · commit c7cde834b081 · 2025-12-10T16:08:44.000+01:00
diff --git a/lib/logstorage/parser.go b/lib/logstorage/parser.go
@@ -1061,7 +1061,7 @@ func (q *Query) GetStatsLabelsAddGroupingByTime(step int64) ([]string, error) {
 				continue
 			}
 			if !p.canReturnLastNResults() {
-				return nil, fmt.Errorf("the pipe `| %q` cannot be put in front of `| %q`, since it modifies or deletes `_time` field", p, ps)
+				return nil, fmt.Errorf("the pipe `| %q` cannot be put in front of `| %q`, since it may modify or delete `_time` field", p, ps)
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -1061,7 +1061,7 @@ func (q *Query) GetStatsLabelsAddGroupingByTime(step int64) ([]string, error) {`
`1061`	`1061`	`continue`
`1062`	`1062`	`}`
`1063`	`1063`	`if !p.canReturnLastNResults() {`
`1064`		- return nil, fmt.Errorf("the pipe `\| %q` cannot be put in front of `\| %q`, since it modifies or deletes `_time` field", p, ps)
	`1064`	+ return nil, fmt.Errorf("the pipe `\| %q` cannot be put in front of `\| %q`, since it may modify or delete `_time` field", p, ps)
`1065`	`1065`	`}`
`1066`	`1066`	`}`
`1067`	`1067`	`}`