Skip to content

Commit

Permalink
[KYUUBI #2006] Improve the readability of ServiceControl list/get ope…
Browse files Browse the repository at this point in the history
…rations

### _Why are the changes needed?_
When the get server and list server commands are displayed, when the namespace exceeds the minimum width, it will be glued to the hostname.

Add a space beyond the minimum cell width in non-verbose mode.

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [x] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #2007 from cxzl25/KYUUBI-2006.

Closes #2006

68bacef [sychen] Add a space beyond the minimum cell width in non-verbose mode

Authored-by: sychen <sychen@trip.com>
Signed-off-by: ulysses-you <ulyssesyou@apache.org>
  • Loading branch information
cxzl25 authored and ulysses-you committed Mar 4, 2022
1 parent 85d515d commit d8ec325
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,9 @@ private[kyuubi] object Tabulator {
for (row <- data) {
for ((cell, i) <- row.zipWithIndex) {
colWidths(i) = math.max(colWidths(i), stringHalfWidth(cell))
if (!verbose) {
colWidths(i) += 1
}
}
}

Expand Down

0 comments on commit d8ec325

Please sign in to comment.