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

Jonathan Eagles commented on TEZ-3724:
--------------------------------------

+1.
Hey [~Sreenath], At first, I made the change the same as you have. Then I 
realised there were some optimizations that could be made, both in readability 
and performance.

I ran the patch 1 through jsperf to get a sense of speed up (roughly 10x, but 
might now matter in this case). Click run tests to measure the difference
https://jsperf.com/jteagles-indexof-vs-match

If you think the improvements are not helpful in this case, we can go with your 
version of the fix. 

> Tez UI on HTTP "corrects" HTTPS REST calls to HTTP
> --------------------------------------------------
>
>                 Key: TEZ-3724
>                 URL: https://issues.apache.org/jira/browse/TEZ-3724
>             Project: Apache Tez
>          Issue Type: Bug
>          Components: UI
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>         Attachments: TEZ-3724.1.patch, TEZ-3724.2.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to