ceph / ceph/go-ceph

Some initial buffer sizes are pretty large

Open
#210 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
691
Forks
296
Avg merge
2d 16h
Merged PRs (30d)
11

Description

Once we have generic retry-resize support across the code in this library we should audit some of the initial sizes such that they make sense as _initial_ sizes.

While working on the RFC for retry support @nixpanic noted the size for the config option buffer was rather large:

Quoth:

This buffer looks pretty large for a configuration option value. Of course this was the earlier default, but starting with 128 bytes would probably be sufficient for many options (booleans, integers, names, ...).

_Originally posted by @nixpanic in https://github.com/ceph/go-ceph/pull/191_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.