[GitHub] [pulsar] tjiuming commented on a diff in pull request #16641: [storage][config][doc] add `skipListSizeLimit` configuration doc of bookkeeper

[GitBox <gi...@apache.org>]

tjiuming commented on code in PR #16641:
URL: https://github.com/apache/pulsar/pull/16641#discussion_r925266175


##########
site2/docs/reference-configuration.md:
##########
@@ -125,6 +125,7 @@ BookKeeper is a replicated log storage system that Pulsar uses for durable stora
 |dbStorage_rocksDB_numLevels||-1|
 |dbStorage_rocksDB_numFilesInLevel0||4|
 |dbStorage_rocksDB_maxSizeInLevel1MB||256|
+|skipListSizeLimit|Heap memory limitation of `EntryMemTable`, The `EntryMemTable` holds in-memory representation to the entries not-yet flushed. When asked to flush, current EntrySkipList is moved to snapshot and is cleared.|64MB|

Review Comment:
   There are `skipListArenaChunkSize` and `skipListArenaMaxAllocSize` in the reference-config, and it's all for `EntryMemTable`, I think whether it's recommended or not, add the configuration into  reference-config is required, in case of someone to ask it.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org