-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Open
Labels
area/adminarea/perfenhancementFeature requests. Not bugs or questions.Feature requests. Not bugs or questions.help wantedNeeds help!Needs help!
Description
/config_dump can produce a remarkably huge response. This will need to be fully buffered both as a protobuf message and an in-memory continuous string. For large configs, this will be slow and may exert memory pressure as well.
See also #31755 as well as prior work streaming admin stats in #19693. This work is not trivial and has risks. See also the difficult attempt to make /stats/prometheus stream as well: #24998 which needed to be rolled back due to complexities in the Prom format that made streaming difficult. See also #28988 which adds a streaming json serializer, which probably should be used for this, rather than creating a protobuf structure and then serializing that.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area/adminarea/perfenhancementFeature requests. Not bugs or questions.Feature requests. Not bugs or questions.help wantedNeeds help!Needs help!