Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18712 )

Change subject: [WIP] KUDU-2671  Make WebUI compatible with custom hash schema
......................................................................


Patch Set 1:

(2 comments)

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

PS1:
style nit: please keep the lines of the description to be 72 characters or 
less; see 
https://git-scm.com/book/en/v2/Distributed-Git-Contributing-to-a-Project#_commit_guidelines
 as linked from 
https://kudu.apache.org/docs/contributing.html#_submitting_patches


http://gerrit.cloudera.org:8080/#/c/18712/1//COMMIT_MSG@13
PS1, Line 13: The custom hash schema if present for a particular range is 
displayed right
            : beside the range schema
I could not see the hash schema as-is there, something that one would expect to 
be similar to the part of the 'Partition Schema' part in the top of the page.  
In particular, there was no information on the number of buckets in each hash 
dimension, just the index of the corresponding bucket.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic8b8d90f70c39f13b838e858c870e08dacbdfcd3
Gerrit-Change-Number: 18712
Gerrit-PatchSet: 1
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Comment-Date: Fri, 08 Jul 2022 17:53:59 +0000
Gerrit-HasComments: Yes

Reply via email to