Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13442 )
Change subject: docs: correct typo in known issues ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13442/1/docs/known_issues.adoc File docs/known_issues.adoc: http://gerrit.cloudera.org:8080/#/c/13442/1/docs/known_issues.adoc@75 PS1, Line 75: column > nit: maybe, use 'cell' here instead? Given that Kudu is a columnar storage >From the other side, if it's a truly columnar store then why there should be >such a restriction if talking, say, about 'cold' columns which are not >involved in majority of queries? Probably, that was about limitations on >primary keys? If so, the most likely this item is outdated and should have been replaced with item at line 49: * Cells making up a composite primary key are limited to a total of 16KB after the internal composite-key encoding done by Kudu. I would consider dropping this item, given that we have the following items listed already: * at line 49 (16KiB for composite primary keys, after encoding) * at line 60 (maximum of 300 columns per table) * at line 70 (64KiB per cell before compression) -- To view, visit http://gerrit.cloudera.org:8080/13442 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9b390e1d3559d827a7081f420ad4eedfbbabdeef Gerrit-Change-Number: 13442 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Alex Rodoni <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 27 May 2019 20:15:08 +0000 Gerrit-HasComments: Yes
