[jira] [Commented] (TEZ-3362) Delete intermediate data at DAG level for Shuffle Handler

2022-03-08 Thread Thomas Graves (Jira)
[ https://issues.apache.org/jira/browse/TEZ-3362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17503138#comment-17503138 ] Thomas Graves commented on TEZ-3362: [~jeagles] [~kshukla]  I know its been a while, I was looking at

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821872406 ## File path: tez-plugins/tez-aux-services/src/test/java/org/apache/tez/auxservices/TestShuffleHandler.java ## @@ -1312,6 +1313,86 @@ protected void

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821871999 ## File path: tez-plugins/tez-aux-services/src/main/java/org/apache/tez/auxservices/ShuffleHandler.java ## @@ -1256,6 +1280,29 @@ private String

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821871579 ## File path: tez-plugins/tez-aux-services/src/main/java/org/apache/tez/auxservices/ShuffleHandler.java ## @@ -1256,6 +1280,29 @@ private String

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821870885 ## File path: tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/VertexImpl.java ## @@ -3758,6 +3780,31 @@ public VertexState transition(VertexImpl

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821870419 ## File path: tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java ## @@ -1772,6 +1774,52 @@ private static void parseVertexEdges(DAGImpl dag,

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821868285 ## File path: tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java ## @@ -883,6 +883,25 @@ public TezConfiguration(boolean loadDefaults) {

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821865132 ## File path: tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java ## @@ -883,6 +883,25 @@ public TezConfiguration(boolean loadDefaults) {

[GitHub] [tez] tez-yetus commented on pull request #190: TEZ-4383: upgrade to mockito 4.3.1

2022-03-08 Thread GitBox
tez-yetus commented on pull request #190: URL: https://github.com/apache/tez/pull/190#issuecomment-1061738481 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 19m 18s |

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821509356 ## File path: tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java ## @@ -883,6 +883,25 @@ public TezConfiguration(boolean loadDefaults) {

[GitHub] [tez] shameersss1 commented on a change in pull request #60: TEZ-3363: Delete intermediate data at the vertex level for Shuffle Handler

2022-03-08 Thread GitBox
shameersss1 commented on a change in pull request #60: URL: https://github.com/apache/tez/pull/60#discussion_r821509356 ## File path: tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java ## @@ -883,6 +883,25 @@ public TezConfiguration(boolean loadDefaults) {