Skip to content

access log: Add ability to batch entries for grpc access log #7674

New issue
New issue
Closed
#7755
Closed
access log: Add ability to batch entries for grpc access log#7674
#7755
Labels
enhancementFeature requests. Not bugs or questions.help wantedNeeds help!
@euroelessar

Description

@euroelessar
euroelessar
opened on Jul 22, 2019

Description:
Currently noticeable chunk of cpu usage (>50%) of grpc access log is spent in syscalls.
This overhead can be eliminated by batching access log entries.

Is it reasonable to extend CommonGrpcAccessLogConfig by two new field: flush_interval and batch_byte_size?

Relevant Links:

envoy/source/extensions/access_loggers/http_grpc/grpc_access_log_impl.cc

Line 406 in c53b12d

// TODO(mattklein123): Consider batching multiple logs and flushing.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementFeature requests. Not bugs or questions.help wantedNeeds help!

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions