Yingchun Lai has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20489 )

Change subject: [server]KUDU-3513 Decrease default value of 
—log_container_preallocate_bytes
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20489/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/20489/1//COMMIT_MSG@12
PS1, Line 12: This might
            : not be significant if we have large number of rows and fewer 
columns but
            : becomes more pronounced in the other case
> The pre-allocation allows for container's data to be more "contiguous" on d
I encountered the same issue about the pre-allocate space when tables have 
large number of columns.
But it's not a good idea to change the default config value IMO, it changes the 
default behavior.



--
To view, visit http://gerrit.cloudera.org:8080/20489
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3dc424f0a6086bb80d904fac94f9327a3bf9423a
Gerrit-Change-Number: 20489
Gerrit-PatchSet: 1
Gerrit-Owner: Abhishek Chennaka <achenn...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Comment-Date: Wed, 11 Oct 2023 03:16:00 +0000
Gerrit-HasComments: Yes

Reply via email to