[
https://issues.apache.org/jira/browse/TEZ-1741?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14204020#comment-14204020
]
Prakash Ramachandran commented on TEZ-1741:
-------------------------------------------
[~Sreenath] This does not apply cleanly on TEZ-8 branch. The variables like
env, application etc that I see in the patch (being modified) seems to be
missing from the TEZ-8 branch.
couple of minor comments
- are the console logs required? seems like left behind from debugging. its
spamming the logs.
- This is a typo?
{code:title=tez-app-configs-controller.js}
configs = configs = this.get('configs').content
{code}
- in loadEntites function why is 'that' being used? shouldn't it be this.key
etc. as its not a callback.
> App view.
> ---------
>
> Key: TEZ-1741
> URL: https://issues.apache.org/jira/browse/TEZ-1741
> Project: Apache Tez
> Issue Type: Sub-task
> Reporter: Sreenath Somarajapuram
> Assignee: Sreenath Somarajapuram
> Attachments: TEZ-1741.1.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
