diff options
| author | Michael Paquier <michael@paquier.xyz> | 2024-07-23 16:55:09 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2024-07-23 16:55:09 +0900 |
| commit | 3b279d89cb5c86fa7ac6faaebb3ddadb2dbe0ca8 (patch) | |
| tree | 7393a15dc085bae735a7ff0728b412492135bd1f /src/backend/access/transam/slru.c | |
| parent | db46016304dc1f9ea402eee6d392f704050467df (diff) | |
Improve comments in slru.{c,h} about segment name format
slru.h described incorrectly how SLRU segment names are formatted
depending on the segment number and if long or short segment names are
used. This commit closes the gap with a better description, fitting
with the reality.
Reported-by: Noah Misch
Author: Aleksander Alekseev
Discussion: https://postgr.es/m/20240626002747.dc.nmisch@google.com
Backpatch-through: 17
Diffstat (limited to 'src/backend/access/transam/slru.c')
| -rw-r--r-- | src/backend/access/transam/slru.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/src/backend/access/transam/slru.c b/src/backend/access/transam/slru.c index 77b05cc0a74..248aa1a6553 100644 --- a/src/backend/access/transam/slru.c +++ b/src/backend/access/transam/slru.c @@ -72,6 +72,21 @@ #include "storage/shmem.h" #include "utils/guc_hooks.h" +/* + * Converts segment number to the filename of the segment. + * + * "path" should point to a buffer at least MAXPGPATH characters long. + * + * If ctl->long_segment_names is true, segno can be in the range [0, 2^60-1]. + * The resulting file name is made of 15 characters, e.g. dir/123456789ABCDEF. + * + * If ctl->long_segment_names is false, segno can be in the range [0, 2^24-1]. + * The resulting file name is made of 4 to 6 characters, as of: + * + * dir/1234 for [0, 2^16-1] + * dir/12345 for [2^16, 2^20-1] + * dir/123456 for [2^20, 2^24-1] + */ static inline int SlruFileName(SlruCtl ctl, char *path, int64 segno) { |