-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Filebeat] aws-s3 - Disable event normalization processing #33673
[Filebeat] aws-s3 - Disable event normalization processing #33673
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
Disable event normalization for the aws-s3 input to reduce allocations when processing events. The input only produces basic types in its events. Either it puts a string into the `message` field or it decodes json into a map[string]interface with encoding/json. Both of those should be fine for the downstream processors and outputs. Relates elastic#33657
9f9c060
to
4ccb09d
Compare
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
This pull request doesn't have a |
@@ -368,3 +373,6 @@ func getProviderFromDomain(endpoint string, ProviderOverride string) string { | |||
} | |||
return "unknown" | |||
} | |||
|
|||
// boolPtr returns a pointer to b. | |||
func boolPtr(b bool) *bool { return &b } |
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.
func boolPtr(b bool) *bool { return &b } | |
func ptr[T any](v bool) *T { return &v } |
🤣
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.
I've got to start thinking more generically. 😆
Disable event normalization for the aws-s3 input to reduce allocations when processing events. The input only produces basic types in its events. Either it puts a string into the `message` field or it decodes json into a map[string]interface with encoding/json. Both of those should be fine for the downstream processors and outputs. Relates #33657
What does this PR do?
Disable event normalization for the aws-s3 input to reduce allocations when processing events. The input only produces basic types in its events. Either it puts a string into the
message
field or it decodes json into a map[string]interface with encoding/json. Both of those should be fine for the downstream processors and outputs.Relates #33657
Why is it important?
It makes Filebeat more efficient.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues
Screenshots