Re: [PR] TEZ-4547: Add Tez AM JobID to the JobConf [tez]

2024-03-13 Thread via GitHub
r0hini commented on code in PR #339: URL: https://github.com/apache/tez/pull/339#discussion_r1523912700 ## tez-mapreduce/src/test/java/org/apache/tez/mapreduce/output/TestMROutput.java: ## @@ -131,6 +133,26 @@ public void testMergeConfig() throws Exception { assertEquals("b

Re: [PR] TEZ-4547: Add Tez AM JobID to the JobConf [tez]

2024-03-13 Thread via GitHub
r0hini commented on code in PR #339: URL: https://github.com/apache/tez/pull/339#discussion_r1523912081 ## tez-mapreduce/src/main/java/org/apache/tez/mapreduce/output/MROutput.java: ## @@ -417,6 +418,7 @@ protected List initializeBase() throws IOException, InterruptedException

Re: [PR] TEZ-4547: Add Tez AM JobID to the JobConf [tez]

2024-03-13 Thread via GitHub
tez-yetus commented on PR #339: URL: https://github.com/apache/tez/pull/339#issuecomment-1995452059 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s | Docker mode

Re: [PR] TEZ-4547: Add Tez AM JobID to the JobConf [tez]

2024-03-13 Thread via GitHub
tez-yetus commented on PR #339: URL: https://github.com/apache/tez/pull/339#issuecomment-1995452045 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 26m 49s | Docker mode a

[jira] [Commented] (TEZ-4547) Add Tez AM JobID to the JobConf

2024-03-13 Thread Venkatasubrahmanian Narayanan (Jira)
[ https://issues.apache.org/jira/browse/TEZ-4547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17826815#comment-17826815 ] Venkatasubrahmanian Narayanan commented on TEZ-4547: Past discussion in

[jira] [Commented] (TEZ-4547) Add Tez AM JobID to the JobConf

2024-03-13 Thread Venkatasubrahmanian Narayanan (Jira)
[ https://issues.apache.org/jira/browse/TEZ-4547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17826814#comment-17826814 ] Venkatasubrahmanian Narayanan commented on TEZ-4547: Github PR: https:/

[PR] TEZ-4547: Add Tez AM JobID to the JobConf [tez]

2024-03-13 Thread via GitHub
VenkatSNarayanan opened a new pull request, #339: URL: https://github.com/apache/tez/pull/339 Some committers require a job-wide UUID to function correctly. Adding the AM JobID to the JobConf will allow applications to pass that to the committers that need it. -- This is an automate

[jira] [Updated] (TEZ-4547) Add Tez AM JobID to the JobConf

2024-03-13 Thread Venkatasubrahmanian Narayanan (Jira)
[ https://issues.apache.org/jira/browse/TEZ-4547?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Venkatasubrahmanian Narayanan updated TEZ-4547: --- Summary: Add Tez AM JobID to the JobConf (was: Add Tez AM JobID to the

[jira] [Created] (TEZ-4547) Add Tez AM JobID to the Configuration object

2024-03-13 Thread Venkatasubrahmanian Narayanan (Jira)
Venkatasubrahmanian Narayanan created TEZ-4547: -- Summary: Add Tez AM JobID to the Configuration object Key: TEZ-4547 URL: https://issues.apache.org/jira/browse/TEZ-4547 Project: Apache Tez

Re: [PR] Bump guava from 31.1-jre to 32.0.0-jre [tez]

2024-03-13 Thread via GitHub
ayushtkn commented on PR #290: URL: https://github.com/apache/tez/pull/290#issuecomment-1994057088 @dependabot rebase -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To uns

Re: [PR] Bump guava from 31.1-jre to 32.0.0-jre [tez]

2024-03-13 Thread via GitHub
dependabot[bot] closed pull request #290: Bump guava from 31.1-jre to 32.0.0-jre URL: https://github.com/apache/tez/pull/290 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] Bump guava from 31.1-jre to 32.0.0-jre [tez]

2024-03-13 Thread via GitHub
dependabot[bot] commented on PR #290: URL: https://github.com/apache/tez/pull/290#issuecomment-1994057731 Looks like com.google.guava:guava is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on