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

Change subject: KUDU-3318 [fs] Add size limit for log block container metadata
......................................................................


Patch Set 8:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/17837/7//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/17837/7//COMMIT_MSG@11
PS7, Line 11: becau
> because
Done


http://gerrit.cloudera.org:8080/#/c/17837/7//COMMIT_MSG@11
PS7, Line 11: . LBM reclaims
            : both .data and corresponding .metadata container files when
            : the .data container file becomes full (i.e. reaches its size
            : or block count threshold).
            : So, the .metadata container file might grow without any limit
> How about:
Done


http://gerrit.cloudera.org:8080/#/c/17837/7//COMMIT_MSG@16
PS7, Line 16: .da
> adds
Done


http://gerrit.cloudera.org:8080/#/c/17837/7//COMMIT_MSG@16
PS7, Line 16: ec
> for
Done


http://gerrit.cloudera.org:8080/#/c/17837/7//COMMIT_MSG@17
PS7, Line 17: This patch adds a size limit for log block container's m
> taking it into account while determining the container's 'full' condition.
Done


http://gerrit.cloudera.org:8080/#/c/17837/7//COMMIT_MSG@17
PS7, Line 17: tadata,
            : taking it into account while determining the container's 'full'
            : condition.
> That gives the LBM an opportunity to reclaim the disk space once the contai
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I12513abf2e45f7bdf091142c31f50d650b6f0cfc
Gerrit-Change-Number: 17837
Gerrit-PatchSet: 8
Gerrit-Owner: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Comment-Date: Sat, 11 Sep 2021 05:50:44 +0000
Gerrit-HasComments: Yes

Reply via email to