[jira] [Commented] (TEZ-3443) Remove a repeated method

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823325#comment-15823325 ] Siddharth Seth commented on TEZ-3443: - [~aplusplus] - the method doesn't provide compatibility with MR.

[jira] [Commented] (TEZ-3529) Tez UI: Add 'All Queries' table in the landing page along 'All DAGs' page

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3529?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823341#comment-15823341 ] Siddharth Seth commented on TEZ-3529: - Thanks for the review [~harishjp] Some more comments on the

[jira] [Commented] (TEZ-3530) Tez UI: Add query details page, and link the page from All Queries table

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823345#comment-15823345 ] Siddharth Seth commented on TEZ-3530: - One question (partly related to the patch). Various columns on

[jira] [Commented] (TEZ-3574) Container reuse won't pickup extra dag level local resource

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823437#comment-15823437 ] Siddharth Seth commented on TEZ-3574: - +1. Patch looks good. Thanks [~aplusplus] > Container reuse

[jira] [Commented] (TEZ-3528) DAGReport progress don't division map and reduce

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823327#comment-15823327 ] Siddharth Seth commented on TEZ-3528: - [~darion] - it looks like this class is not used anywhere useful,

[jira] [Commented] (TEZ-3498) Tez UI: Optimize display of Task details page

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823336#comment-15823336 ] Siddharth Seth commented on TEZ-3498: - bq. This patch was created with the assumption that the attempts

[jira] [Comment Edited] (TEZ-3530) Tez UI: Add query details page, and link the page from All Queries table

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823342#comment-15823342 ] Siddharth Seth edited comment on TEZ-3530 at 1/16/17 1:28 AM: -- +1. Looks good.

[jira] [Commented] (TEZ-3530) Tez UI: Add query details page, and link the page from All Queries table

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823342#comment-15823342 ] Siddharth Seth commented on TEZ-3530: - +1. Looks good. Thanks [~Sreenath], [~harishjp] for the review.

[jira] [Commented] (TEZ-3458) Auto grouping for cartesian product edge(unpartitioned case)

2017-01-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823432#comment-15823432 ] Siddharth Seth commented on TEZ-3458: - [~mingma] - would be great if you could review the updated patch.

[jira] [Commented] (TEZ-3530) Tez UI: Add query details page, and link the page from All Queries table

2017-01-15 Thread Sreenath Somarajapuram (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-3530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15823423#comment-15823423 ] Sreenath Somarajapuram commented on TEZ-3530: - bq. One question (partly related to the patch).