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

Alejandro Abdelnur commented on OOZIE-887:
------------------------------------------

looks good, just a couple of final tweaks:

* there are a few lines (introduced) by the patch that are going well beyond 
132 chars, please reformat those.
* OozieCLI & BaseAdminServlet when looping through the avail TZ, you are using 
continue to skip some processing. would you modify the IF CONDITION so the 
logic to exclude is within the IF BLOCK, then we don't use continue (it will be 
easier to follow).
                
> Support for choosing timezone in Oozie UI
> -----------------------------------------
>
>                 Key: OOZIE-887
>                 URL: https://issues.apache.org/jira/browse/OOZIE-887
>             Project: Oozie
>          Issue Type: New Feature
>            Reporter: Robert Kanter
>            Assignee: Robert Kanter
>         Attachments: OOZIE-887-v2.patch, OOZIE-887-v3.patch, 
> OOZIE-887-v4.patch, OOZIE-887.patch
>
>
> Add the ability to choose a different timezone (e.g. PST) in the Oozie web UI 
> and command line.  This would only be a superficial change that only affects 
> the web UI and command line output; it wouldn't change any of the actual 
> processing, logs, etc.  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to