You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Will Berkeley (Code Review)" <ge...@cloudera.org> on 2017/06/09 17:54:56 UTC

[kudu-CR] [webui] Fix placement of tombstone tablet blurb

Will Berkeley has uploaded a new change for review.

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

Change subject: [webui] Fix placement of tombstone tablet blurb
......................................................................

[webui] Fix placement of tombstone tablet blurb

In b492d8ed3c, some text explaining what a tombstoned tablet is
was misplaced and ended up below the rest of the tombstoned
tablets section. This change puts it back in the correct place,
just underneath the "Tombstoned tablets" header for that
section.

Change-Id: I635c5119d1c6e223e2db2f73b2c7f47ce4bdb0a5
---
M src/kudu/tserver/tserver-path-handlers.cc
1 file changed, 6 insertions(+), 6 deletions(-)


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

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

[kudu-CR] [webui] Fix placement of tombstone tablet blurb

Posted by "Dan Burkert (Code Review)" <ge...@cloudera.org>.
Dan Burkert has posted comments on this change.

Change subject: [webui] Fix placement of tombstone tablet blurb
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I635c5119d1c6e223e2db2f73b2c7f47ce4bdb0a5
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Will Berkeley <wd...@gmail.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No

[kudu-CR] [webui] Fix placement of tombstone tablet blurb

Posted by "Dan Burkert (Code Review)" <ge...@cloudera.org>.
Dan Burkert has posted comments on this change.

Change subject: [webui] Fix placement of tombstone tablet blurb
......................................................................


Patch Set 2: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I635c5119d1c6e223e2db2f73b2c7f47ce4bdb0a5
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Will Berkeley <wd...@gmail.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No

[kudu-CR] [webui] Fix placement of tombstone tablet blurb

Posted by "Will Berkeley (Code Review)" <ge...@cloudera.org>.
Will Berkeley has submitted this change and it was merged.

Change subject: [webui] Fix placement of tombstone tablet blurb
......................................................................


[webui] Fix placement of tombstone tablet blurb

In b492d8ed3c, some text explaining what a tombstoned tablet is
was misplaced and ended up below the rest of the tombstoned
tablets section. This change puts it back in the correct place,
just underneath the "Tombstoned tablets" header for that
section.

Change-Id: I635c5119d1c6e223e2db2f73b2c7f47ce4bdb0a5
Reviewed-on: http://gerrit.cloudera.org:8080/7136
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <da...@apache.org>
---
M src/kudu/tserver/tserver-path-handlers.cc
1 file changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Dan Burkert: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I635c5119d1c6e223e2db2f73b2c7f47ce4bdb0a5
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Will Berkeley <wd...@gmail.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>