You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Will Berkeley (Code Review)" <ge...@cloudera.org> on 2016/05/11 06:13:24 UTC

[kudu-CR] KUDU-1388: Master web UI should show table's partition schema

Will Berkeley has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3022

Change subject: KUDU-1388: Master web UI should show table's partition schema
......................................................................

KUDU-1388: Master web UI should show table's partition schema

This patch adds PartitionSchema::DisplayString that creates a pretty-printed
string suitable for display, and changes the web UI to display this string on
/table.

The format is

Hash bucket schemas:
  Key columns:
    name_s00 type_s00
    name_s01 type_s01
    ...

  Key columns:
    name_s10 type_s10
    name_s11 type_s11
    ...

  ...

Range columns:
  name_r0 type_r0
  name_r1 type_r1

The display on the web UI is naive and it won't, e.g., display Unicode
characters correctly, but that's true of the web UI in general and a fix
should probably wait until after KUDU-981.

Change-Id: I81b634f6dd51e63205bbd7956273b74ca63459f7
---
M src/kudu/common/partition.cc
M src/kudu/common/partition.h
M src/kudu/master/master-path-handlers.cc
3 files changed, 38 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/3022/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3022
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I81b634f6dd51e63205bbd7956273b74ca63459f7
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Will Berkeley <wd...@gmail.com>