-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split plugin #299
Split plugin #299
Conversation
2d69deb
to
7ec3fd9
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #299 +/- ##
==========================================
+ Coverage 67.86% 68.72% +0.85%
==========================================
Files 104 108 +4
Lines 10705 10960 +255
==========================================
+ Hits 7265 7532 +267
+ Misses 2977 2955 -22
- Partials 463 473 +10 ☔ View full report in Codecov by Sentry. |
45928a6
to
2e0038d
Compare
2e0038d
to
ab4592a
Compare
aaf5e69
to
2f675a5
Compare
aed744a
to
cd0d309
Compare
cd0d309
to
263371a
Compare
6e29172
to
a77bd3a
Compare
18432e0
to
f351d33
Compare
f351d33
to
3d9f55e
Compare
b7145c6
to
58eaee1
Compare
58eaee1
to
b280f9a
Compare
plugin/action/remap/remap.go
Outdated
func (p *Plugin) Do(event *pipeline.Event) pipeline.ActionResult { | ||
node := event.Root.Dig(p.config.Field_...) | ||
if !node.IsArray() { | ||
p.logger.Warn("skip an event because is not an array", zap.String("type", node.TypeStr())) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because field is not an array
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
b280f9a
to
f352d1b
Compare
Fixes #293