Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11091 )

Change subject: Expose the block manager in ReadableBlocks
......................................................................


Patch Set 1:

(1 comment)

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

http://gerrit.cloudera.org:8080/#/c/11091/1//COMMIT_MSG@9
PS1, Line 9: This is a preliminary patch that could be useful in
           : the fix for KUDU-2469.
> It's hard to imagine how you'd have access to the ReadableBlock but not to
One example is that CFileReaders take ownership of ReadableBlocks and have no 
knowledge of the rest of the FS layer.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icc346177b29cda7a49a83cedfbd5fec2962d330f
Gerrit-Change-Number: 11091
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Comment-Date: Tue, 31 Jul 2018 18:30:45 +0000
Gerrit-HasComments: Yes

Reply via email to