[ 
https://issues.apache.org/jira/browse/HBASE-20577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16478073#comment-16478073
 ] 

Ted Yu commented on HBASE-20577:
--------------------------------

There was one hunk rejected in 
hbase-http/src/main/java/org/apache/hadoop/hbase/http/log/LogLevel.java

Please rebase

The patch looks good.

> Make Log Level page design consistent with the design of other pages in UI
> --------------------------------------------------------------------------
>
>                 Key: HBASE-20577
>                 URL: https://issues.apache.org/jira/browse/HBASE-20577
>             Project: HBase
>          Issue Type: Improvement
>          Components: UI, Usability
>            Reporter: Nihal Jain
>            Assignee: Nihal Jain
>            Priority: Major
>             Fix For: 3.0.0
>
>         Attachments: HBASE-20577.master.001.patch, 
> HBASE-20577.master.002.patch, after_patch_LogLevel_CLI.png, 
> after_patch_get_log_level.png, after_patch_require_field_validation.png, 
> after_patch_set_log_level_bad.png, after_patch_set_log_level_success.png, 
> before_patch_no_validation_required_field.png
>
>
> The Log Level page in web UI seems out of the place. I think we should make 
> it look consistent with design of other pages in HBase web UI.
> Also, validation of required fields should be done, otherwise user should not 
> be allowed to click submit button.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to