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

ASF GitHub Bot commented on STORM-1254:
---------------------------------------

Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1139#discussion_r54165835
  
    --- Diff: storm-core/src/clj/org/apache/storm/ui/core.clj ---
    @@ -134,14 +136,16 @@
     
     (defn logviewer-link [host fname secure?]
       (if (and secure? (*STORM-CONF* LOGVIEWER-HTTPS-PORT))
    -    (url-format "https://%s:%s/log?file=%s";
    -      host
    -      (*STORM-CONF* LOGVIEWER-HTTPS-PORT)
    -      fname)
    -    (url-format "http://%s:%s/log?file=%s";
    -      host
    -      (*STORM-CONF* LOGVIEWER-PORT)
    -      fname)))
    +    (UIHelpers/urlFormat "https://%s:%s/log?file=%s";
    +      (to-array
    +        [host
    +        (*STORM-CONF* LOGVIEWER-HTTPS-PORT)
    +        fname]))
    +    (UIHelpers/urlFormat "http://%s:%s/log?file=%s";
    +      (to-array
    +        [host
    +        (*STORM-CONF* LOGVIEWER-PORT)
    +        fname]))))
    --- End diff --
    
    This is the line that I was getting the exception on.  I will investigate a 
bit more to see if I can understand what is happening.


> port backtype.storm.ui.helpers to java
> --------------------------------------
>
>                 Key: STORM-1254
>                 URL: https://issues.apache.org/jira/browse/STORM-1254
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Jark Wu
>              Labels: java-migration, jstorm-merger
>
> Helper functions for the UI/HTTP



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to