Re: [PR] [WIP] TEZ-4569: SCATTER_GATHER + BROADCAST hangs on DAG Recovery [tez]

2024-06-12 Thread via GitHub
tez-yetus commented on PR #361: URL: https://github.com/apache/tez/pull/361#issuecomment-2163421264 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 32s | Docker mode

Re: [PR] [WIP] TEZ-4569: SCATTER_GATHER + BROADCAST hangs on DAG Recovery [tez]

2024-06-12 Thread via GitHub
tez-yetus commented on PR #361: URL: https://github.com/apache/tez/pull/361#issuecomment-2162620410 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 27m 8s | Docker mode

Re: [PR] TEZ-4568: ProfileServlet: add html to output formats and prepare for profiler 3.0 [tez]

2024-06-06 Thread via GitHub
abstractdog merged PR #360: URL: https://github.com/apache/tez/pull/360 -- 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 unsubscribe, e-mail:

Re: [PR] TEZ-4568: ProfileServlet: add html to output formats and prepare for profiler 3.0 [tez]

2024-06-06 Thread via GitHub
tez-yetus commented on PR #360: URL: https://github.com/apache/tez/pull/360#issuecomment-2151922918 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 9s | Docker mode

Re: [PR] TEZ-4568: ProfileServlet: add html to output formats and prepare for profiler 3.0 [tez]

2024-06-06 Thread via GitHub
tez-yetus commented on PR #360: URL: https://github.com/apache/tez/pull/360#issuecomment-2151843267 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | Docker mode

Re: [PR] TEZ-4568: ProfileServlet: add html to output formats and prepare for profiler 3.0 [tez]

2024-06-06 Thread via GitHub
tez-yetus commented on PR #360: URL: https://github.com/apache/tez/pull/360#issuecomment-2151843228 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 8s | Docker mode

Re: [PR] TEZ-4568: ProfileServlet: add html to output formats [tez]

2024-06-05 Thread via GitHub
tez-yetus commented on PR #360: URL: https://github.com/apache/tez/pull/360#issuecomment-2150466903 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 13m 42s | Docker mode

[PR] TEZ-4568: ProfileServlet: add html to output formats [tez]

2024-06-05 Thread via GitHub
abstractdog opened a new pull request, #360: URL: https://github.com/apache/tez/pull/360 (no comment) -- 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 unsubscribe,

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

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

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

2024-05-28 Thread via GitHub
VenkatSNarayanan commented on code in PR #339: URL: https://github.com/apache/tez/pull/339#discussion_r1617772403 ## tez-mapreduce/src/main/java/org/apache/tez/mapreduce/hadoop/MRJobConfig.java: ## @@ -131,6 +131,11 @@ public interface MRJobConfig { public static final

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

2024-05-28 Thread via GitHub
VenkatSNarayanan commented on code in PR #339: URL: https://github.com/apache/tez/pull/339#discussion_r1617772160 ## tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/OutputCommitterContextImpl.java: ## @@ -34,14 +34,16 @@ public class OutputCommitterContextImpl implements

Re: [PR] TEZ-4508: Allow the FAIR_PARALLELISM mode to accept multiple source vertices [tez]

2024-05-28 Thread via GitHub
okumin closed pull request #306: TEZ-4508: Allow the FAIR_PARALLELISM mode to accept multiple source vertices URL: https://github.com/apache/tez/pull/306 -- 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

Re: [PR] TEZ-4508: Allow the FAIR_PARALLELISM mode to accept multiple source vertices [tez]

2024-05-28 Thread via GitHub
okumin commented on PR #306: URL: https://github.com/apache/tez/pull/306#issuecomment-2135214690 While testing various cases, I found this might not work when amounts of unified vertices are uneven. I will close this one once.

Re: [PR] Bump dependency-check-maven from 1.3.6 to 3.2.0 [tez]

2024-05-28 Thread via GitHub
tez-yetus commented on PR #252: URL: https://github.com/apache/tez/pull/252#issuecomment-2134620223 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 22m 19s | Docker mode

Re: [PR] Bump dependency-check-maven from 1.3.6 to 3.2.0 [tez]

2024-05-28 Thread via GitHub
ayushtkn commented on PR #252: URL: https://github.com/apache/tez/pull/252#issuecomment-2134403820 @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

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-27 Thread via GitHub
ayushtkn merged PR #356: URL: https://github.com/apache/tez/pull/356 -- 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 unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

Re: [PR] TEZ-4355: Unit test precommit improvements - full coverage [tez]

2024-05-27 Thread via GitHub
tez-yetus commented on PR #359: URL: https://github.com/apache/tez/pull/359#issuecomment-2134065949 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 9s | Docker mode

Re: [PR] TEZ-4355: Unit test precommit improvements - full coverage [tez]

2024-05-27 Thread via GitHub
tez-yetus commented on PR #359: URL: https://github.com/apache/tez/pull/359#issuecomment-2133991375 (!) A patch to the testing environment has been detected. Re-executing against the patched versions to perform further tests. The console is at

Re: [PR] TEZ-4355: Unit test precommit improvements - full coverage [tez]

2024-05-27 Thread via GitHub
tez-yetus commented on PR #359: URL: https://github.com/apache/tez/pull/359#issuecomment-2133980639 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 9s | Docker mode

Re: [PR] TEZ-4355: Unit test precommit improvements - full coverage [tez]

2024-05-27 Thread via GitHub
tez-yetus commented on PR #359: URL: https://github.com/apache/tez/pull/359#issuecomment-2133979454 (!) A patch to the testing environment has been detected. Re-executing against the patched versions to perform further tests. The console is at

Re: [PR] TEZ-4355: Unit test precommit improvements - full coverage [tez]

2024-05-27 Thread via GitHub
tez-yetus commented on PR #359: URL: https://github.com/apache/tez/pull/359#issuecomment-2133967579 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 12m 55s | Docker mode

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-27 Thread via GitHub
tez-yetus commented on PR #356: URL: https://github.com/apache/tez/pull/356#issuecomment-2133607294 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 26m 42s | Docker mode

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-27 Thread via GitHub
abstractdog commented on code in PR #356: URL: https://github.com/apache/tez/pull/356#discussion_r1615656673 ## tez-dag/src/main/java/org/apache/tez/client/LocalClient.java: ## @@ -204,6 +206,9 @@ public ApplicationReport getApplicationReport(ApplicationId appId) {

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-27 Thread via GitHub
abstractdog commented on code in PR #356: URL: https://github.com/apache/tez/pull/356#discussion_r1615644419 ## tez-api/src/main/java/org/apache/tez/client/TezClient.java: ## @@ -1286,4 +1286,12 @@ public static ApplicationId appIdfromString(String appIdStr) { +

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-22 Thread via GitHub
ayushtkn commented on code in PR #356: URL: https://github.com/apache/tez/pull/356#discussion_r1610382381 ## tez-dag/src/main/java/org/apache/tez/client/LocalClient.java: ## @@ -204,6 +206,9 @@ public ApplicationReport getApplicationReport(ApplicationId appId) {

Re: [PR] TEZ-4540: Reading proto data more than 2GB from multiple splits fails [tez]

2024-05-22 Thread via GitHub
Aggarwal-Raghav commented on PR #334: URL: https://github.com/apache/tez/pull/334#issuecomment-2124978152 @abstractdog, can you please help with the review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-22 Thread via GitHub
abstractdog commented on PR #356: URL: https://github.com/apache/tez/pull/356#issuecomment-2124589652 @ayushtkn , @jteagles : could you take a look at this one? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] TEZ-4566: NPE in TezChild while fetching attemptId when container is asked to shut down [tez]

2024-05-22 Thread via GitHub
ayushtkn merged PR #358: URL: https://github.com/apache/tez/pull/358 -- 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 unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

Re: [PR] TEZ-4566: NPE in TezChild while fetching attemptId when container is asked to shut down [tez]

2024-05-21 Thread via GitHub
ayushtkn commented on PR #358: URL: https://github.com/apache/tez/pull/358#issuecomment-2123380280 Ran the tests locally **tez-dag** ``` INFO] --- surefire:3.0.0-M4:test (default-test) @ tez-dag --- [INFO] [INFO] ---

Re: [PR] TEZ-4566: NPE in TezChild while fetching attemptId when container is asked to shut down [tez]

2024-05-21 Thread via GitHub
ayushtkn commented on PR #358: URL: https://github.com/apache/tez/pull/358#issuecomment-2123329928 One one of the LOG lines are there ``` 2024-05-21T12:41:38,884 INFO [TezChild] app.TezTaskCommunicatorImpl: Container with id: container_1716320479875_0001_00_24 is valid, but no

Re: [PR] TEZ-4566: NPE in TezChild while fetching attemptId when container is asked to shut down [tez]

2024-05-21 Thread via GitHub
abstractdog commented on PR #358: URL: https://github.com/apache/tez/pull/358#issuecomment-2123234816 this is a very weird edge case and I'm shocked we haven't bumped into it more often the whole, moved block is useless when the container should die so this patch makes sense this

Re: [PR] TEZ-4566: NPE in TezChild while fetching attemptId. [tez]

2024-05-21 Thread via GitHub
tez-yetus commented on PR #358: URL: https://github.com/apache/tez/pull/358#issuecomment-2123080221 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 24m 49s | Docker mode

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-17 Thread via GitHub
tez-yetus commented on PR #356: URL: https://github.com/apache/tez/pull/356#issuecomment-2117015926 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 36s | Docker mode

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-17 Thread via GitHub
tez-yetus commented on PR #356: URL: https://github.com/apache/tez/pull/356#issuecomment-2116941635 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 33s | Docker mode

Re: [PR] TEZ-4565. TestAnalyzer subtest testInternalPreemption is flaky [tez]

2024-05-17 Thread via GitHub
abstractdog commented on PR #357: URL: https://github.com/apache/tez/pull/357#issuecomment-2116938810 I'm actually fine with the patch, however, I'm a bit struggling to understand the cause this unit test looks like describing an exact scenario, however the underlying taskattempt ids

Re: [PR] TEZ-4565. TestAnalyzer subtest testInternalPreemption is flaky [tez]

2024-05-16 Thread via GitHub
jteagles commented on PR #357: URL: https://github.com/apache/tez/pull/357#issuecomment-2116439843 @abstractdog , left this as two commits so you can see the first failed and the second succeeds. Will need squash and merge. -- This is an automated message from the Apache Git Service. To

Re: [PR] TEZ-4565. TestAnalyzer subtest testInternalPreemption is flaky [tez]

2024-05-16 Thread via GitHub
tez-yetus commented on PR #357: URL: https://github.com/apache/tez/pull/357#issuecomment-2116424132 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 23m 11s | Docker mode

Re: [PR] TEZ-4565. TestAnalyzer subtest testInternalPreemption is flaky [tez]

2024-05-16 Thread via GitHub
jteagles commented on PR #357: URL: https://github.com/apache/tez/pull/357#issuecomment-2116380881 Second commit is try to fix the flaky test -- 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

Re: [PR] TEZ-4565. TestAnalyzer subtest testInternalPreemption is flaky [tez]

2024-05-16 Thread via GitHub
tez-yetus commented on PR #357: URL: https://github.com/apache/tez/pull/357#issuecomment-2116259504 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 22m 21s | Docker mode

Re: [PR] TEZ-4565. TestAnalyzer subtest testInternalPreemption is flaky [tez]

2024-05-16 Thread via GitHub
jteagles commented on PR #357: URL: https://github.com/apache/tez/pull/357#issuecomment-2116200424 This is just a test patch to induce the flaky test -- 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

[PR] TEZ-4565. TestAnalyzer subtest testInternalPreemption is flaky [tez]

2024-05-16 Thread via GitHub
jteagles opened a new pull request, #357: URL: https://github.com/apache/tez/pull/357 Occasionally, attempt comes back as 01_2 and doesn't match -- 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

Re: [PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-16 Thread via GitHub
tez-yetus commented on PR #356: URL: https://github.com/apache/tez/pull/356#issuecomment-2115773136 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 26m 35s | Docker mode

[PR] TEZ-4564: TezClient to expose Tez AM host:port [tez]

2024-05-16 Thread via GitHub
abstractdog opened a new pull request, #356: URL: https://github.com/apache/tez/pull/356 (no comment) -- 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 unsubscribe,

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

2024-05-14 Thread via GitHub
abstractdog commented on code in PR #339: URL: https://github.com/apache/tez/pull/339#discussion_r1599582588 ## tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/OutputCommitterContextImpl.java: ## @@ -34,14 +34,16 @@ public class OutputCommitterContextImpl implements

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

2024-05-14 Thread via GitHub
abstractdog commented on code in PR #339: URL: https://github.com/apache/tez/pull/339#discussion_r1599573887 ## tez-mapreduce/src/main/java/org/apache/tez/mapreduce/hadoop/MRJobConfig.java: ## @@ -131,6 +131,11 @@ public interface MRJobConfig { public static final String

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandler instead of LocalDirAllocator [tez]

2024-05-14 Thread via GitHub
abstractdog merged PR #355: URL: https://github.com/apache/tez/pull/355 -- 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 unsubscribe, e-mail:

Re: [PR] TEZ-4542: Tez application may fail due to int overflow when record size is large and sort memory is low. [tez]

2024-05-14 Thread via GitHub
abstractdog merged PR #336: URL: https://github.com/apache/tez/pull/336 -- 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 unsubscribe, e-mail:

Re: [PR] TEZ-4542: Tez application may fail due to int overflow when record size is large and sort memory is low. [tez]

2024-05-14 Thread via GitHub
rbalamohan commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-2109427570 LGTM. +1. BTW, pipelinesorter is the brainchild of @t3rmin4t0r . :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandler instead of LocalDirAllocator [tez]

2024-05-13 Thread via GitHub
jteagles commented on PR #355: URL: https://github.com/apache/tez/pull/355#issuecomment-2108763375 > > @abstractdog , do you know who might be good to review this ShuffleHandler change? Handles cases where drives go bad or are added back after being fixed > > thanks for this patch

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandler instead of LocalDirAllocator [tez]

2024-05-13 Thread via GitHub
abstractdog commented on PR #355: URL: https://github.com/apache/tez/pull/355#issuecomment-2108641377 > @abstractdog , do you know who might be good to review this ShuffleHandler change? Handles cases where drives go bad or are added back after being fixed thanks for this patch

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandler instead of LocalDirAllocator [tez]

2024-05-13 Thread via GitHub
jteagles commented on PR #355: URL: https://github.com/apache/tez/pull/355#issuecomment-2108066421 Checkstyle issues have been addressed. -- 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

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandler instead of LocalDirAllocator [tez]

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

Re: [PR] TEZ-4542: Tez application may fail due to int overflow when record size is large and sort memory is low. [tez]

2024-05-13 Thread via GitHub
abstractdog commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-2106927518 the affected lines were touch by @rbalamohan in the past frequently, he is the pipelined sorted expert, is there a chance you can take a look on this tiny patch? -- This is an automated

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandler instead of LocalDirAllocator [tez]

2024-05-10 Thread via GitHub
jteagles commented on PR #355: URL: https://github.com/apache/tez/pull/355#issuecomment-2105397956 @abstractdog , do you know who might be good to review this ShuffleHandler change? Handles cases where drives go bad or are added back after being fixed -- This is an automated message from

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandler instead of LocalDirAllocator [tez]

2024-05-10 Thread via GitHub
tez-yetus commented on PR #355: URL: https://github.com/apache/tez/pull/355#issuecomment-2105348233 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 8s | Docker mode

Re: [PR] TEZ-4563: Bump org.bouncycastle:bcprov-jdk18on from 1.77 to 1.78 [tez]

2024-05-09 Thread via GitHub
ayushtkn merged PR #352: URL: https://github.com/apache/tez/pull/352 -- 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 unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

Re: [PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandle… [tez]

2024-05-09 Thread via GitHub
tez-yetus commented on PR #355: URL: https://github.com/apache/tez/pull/355#issuecomment-2103573620 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 14m 12s | Docker mode

[PR] [TEZ-4019] Modify Tez shuffle handler to use AuxiliaryLocalPathHandle… [tez]

2024-05-09 Thread via GitHub
jteagles opened a new pull request, #355: URL: https://github.com/apache/tez/pull/355 …r instead of LocalDirAllocator -- 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] TEZ-4542: Tez application may fail due to int overflow when record size is large and sort memory is low. [tez]

2024-05-08 Thread via GitHub
zhengchenyu commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-2101818613 @abstractdog Can you please review this PR? -- 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

Re: [PR] TEZ-4563: Bump org.bouncycastle:bcprov-jdk18on from 1.77 to 1.78 [tez]

2024-05-08 Thread via GitHub
tez-yetus commented on PR #352: URL: https://github.com/apache/tez/pull/352#issuecomment-2099843076 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 39s | Docker mode

Re: [PR] Bump jettison from 1.5.3 to 1.5.4 [tez]

2024-05-07 Thread via GitHub
dependabot[bot] commented on PR #275: URL: https://github.com/apache/tez/pull/275#issuecomment-2099737250 Looks like org.codehaus.jettison:jettison 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,

Re: [PR] Bump jettison from 1.5.3 to 1.5.4 [tez]

2024-05-07 Thread via GitHub
dependabot[bot] closed pull request #275: Bump jettison from 1.5.3 to 1.5.4 URL: https://github.com/apache/tez/pull/275 -- 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 jettison from 1.5.3 to 1.5.4 [tez]

2024-05-07 Thread via GitHub
ayushtkn commented on PR #275: URL: https://github.com/apache/tez/pull/275#issuecomment-2099736454 @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

Re: [PR] Bump protobuf-java from 3.21.1 to 3.21.7 [tez]

2024-05-07 Thread via GitHub
dependabot[bot] closed pull request #240: Bump protobuf-java from 3.21.1 to 3.21.7 URL: https://github.com/apache/tez/pull/240 -- 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.

Re: [PR] Bump protobuf-java from 3.21.1 to 3.21.7 [tez]

2024-05-07 Thread via GitHub
dependabot[bot] commented on PR #240: URL: https://github.com/apache/tez/pull/240#issuecomment-2099734953 Looks like com.google.protobuf:protobuf-java 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,

Re: [PR] Bump protobuf-java from 3.21.1 to 3.21.7 [tez]

2024-05-07 Thread via GitHub
ayushtkn commented on PR #240: URL: https://github.com/apache/tez/pull/240#issuecomment-2099733868 @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

Re: [PR] Bump org.bouncycastle:bcprov-jdk18on from 1.77 to 1.78 [tez]

2024-05-07 Thread via GitHub
ayushtkn commented on PR #352: URL: https://github.com/apache/tez/pull/352#issuecomment-2099728554 @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

Re: [PR] TEZ-4562. Fix Tez Job Analyzer after TEZ_DAG_EXTRA_INFO [tez]

2024-05-07 Thread via GitHub
tez-yetus commented on PR #354: URL: https://github.com/apache/tez/pull/354#issuecomment-2099433987 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 28m 21s | Docker mode

[PR] TEZ-4562. Fix Tez Job Analyzer after TEZ_DAG_EXTRA_INFO [tez]

2024-05-07 Thread via GitHub
jteagles opened a new pull request, #354: URL: https://github.com/apache/tez/pull/354 Not ready to commit yet. -- 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] TEZ-4547: Add Tez AM JobID to the JobConf [tez]

2024-05-07 Thread via GitHub
VenkatSNarayanan commented on PR #339: URL: https://github.com/apache/tez/pull/339#issuecomment-2099211212 @abstractdog @shameersss1 Is there anything else needed? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] TEZ-4559: Fix Retry logic in case of Recovery [tez]

2024-05-07 Thread via GitHub
ayushtkn merged PR #353: URL: https://github.com/apache/tez/pull/353 -- 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 unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

Re: [PR] TEZ-4550. Upgrade commons-cli to 1.6.0. [tez]

2024-05-06 Thread via GitHub
slfan1989 commented on PR #343: URL: https://github.com/apache/tez/pull/343#issuecomment-2097149662 > @slfan1989 : can you please check if TestTezJobs failures are related? the rest is TEZ-4559 Thank you for the review! I will follow up and check the reasons for the failure of

Re: [PR] TEZ-4559: Fix Retry logic in case of Recovery [tez]

2024-05-06 Thread via GitHub
tez-yetus commented on PR #353: URL: https://github.com/apache/tez/pull/353#issuecomment-2096068702 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 5s | Docker mode

Re: [PR] TEZ-4559: Fix Retry logic in case of Recovery [tez]

2024-05-06 Thread via GitHub
tez-yetus commented on PR #353: URL: https://github.com/apache/tez/pull/353#issuecomment-209559 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 56m 30s | Docker mode

Re: [PR] Bump org.bouncycastle:bcprov-jdk18on from 1.77 to 1.78 [tez]

2024-05-06 Thread via GitHub
tez-yetus commented on PR #352: URL: https://github.com/apache/tez/pull/352#issuecomment-2095579935 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 26m 16s | Docker mode

Re: [PR] TEZ-4552. Upgrade protobuf to 3.24.4 due to CVE. [tez]

2024-05-06 Thread via GitHub
abstractdog merged PR #345: URL: https://github.com/apache/tez/pull/345 -- 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 unsubscribe, e-mail:

Re: [PR] TEZ-4552. Upgrade protobuf to 3.24.4 due to CVE. [tez]

2024-05-06 Thread via GitHub
abstractdog commented on PR #345: URL: https://github.com/apache/tez/pull/345#issuecomment-2095387494 failing unit tests are because of [TEZ-4559](https://issues.apache.org/jira/browse/TEZ-4559), this looks good to me -- This is an automated message from the Apache Git Service. To

Re: [PR] TEZ-4550. Upgrade commons-cli to 1.6.0. [tez]

2024-05-06 Thread via GitHub
abstractdog commented on PR #343: URL: https://github.com/apache/tez/pull/343#issuecomment-2095387023 @slfan1989 : can you please check if TestTezJobs failures are related? the rest is TEZ-4559 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] TEZ-4560. Upgrade bouncycastle to 1.77 due to CVE. [tez]

2024-05-06 Thread via GitHub
abstractdog merged PR #351: URL: https://github.com/apache/tez/pull/351 -- 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 unsubscribe, e-mail:

[PR] Bump org.bouncycastle:bcprov-jdk18on from 1.77 to 1.78 [tez]

2024-05-06 Thread via GitHub
dependabot[bot] opened a new pull request, #352: URL: https://github.com/apache/tez/pull/352 Bumps [org.bouncycastle:bcprov-jdk18on](https://github.com/bcgit/bc-java) from 1.77 to 1.78. Changelog Sourced from

Re: [PR] TEZ-4560. Upgrade bouncycastle to 1.77 due to CVE. [tez]

2024-05-06 Thread via GitHub
abstractdog commented on PR #351: URL: https://github.com/apache/tez/pull/351#issuecomment-2095384223 failing unit tests are because of [TEZ-4559](https://issues.apache.org/jira/browse/TEZ-4559), this looks good to me -- This is an automated message from the Apache Git Service. To

Re: [PR] TEZ-4551. Upgrade commons-io to 2.16.0. [tez]

2024-05-06 Thread via GitHub
abstractdog merged PR #344: URL: https://github.com/apache/tez/pull/344 -- 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 unsubscribe, e-mail:

Re: [PR] TEZ-4551. Upgrade commons-io to 2.16.0. [tez]

2024-05-06 Thread via GitHub
abstractdog commented on PR #344: URL: https://github.com/apache/tez/pull/344#issuecomment-2095381682 failing unit tests are because of [TEZ-4559](https://issues.apache.org/jira/browse/TEZ-4559), this looks good to me -- This is an automated message from the Apache Git Service. To

Re: [PR] TEZ-4560. Upgrade bouncycastle to 1.77 due to CVE. [tez]

2024-05-04 Thread via GitHub
tez-yetus commented on PR #351: URL: https://github.com/apache/tez/pull/351#issuecomment-2094644056 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 12m 23s | Docker mode

Re: [PR] TEZ-4551. Upgrade commons-io to 2.16.0. [tez]

2024-05-04 Thread via GitHub
tez-yetus commented on PR #344: URL: https://github.com/apache/tez/pull/344#issuecomment-2094636658 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 25m 16s | Docker mode

Re: [PR] TEZ-4550. Upgrade commons-cli to 1.6.0. [tez]

2024-05-04 Thread via GitHub
tez-yetus commented on PR #343: URL: https://github.com/apache/tez/pull/343#issuecomment-2094628701 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 13s | Docker mode

Re: [PR] TEZ-4552. Upgrade protobuf to 3.24.4 due to CVE. [tez]

2024-05-04 Thread via GitHub
tez-yetus commented on PR #345: URL: https://github.com/apache/tez/pull/345#issuecomment-2094570613 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 22m 11s | Docker mode

Re: [PR] TEZ-4552. Upgrade protobuf to 3.24.4. [tez]

2024-05-04 Thread via GitHub
tez-yetus commented on PR #345: URL: https://github.com/apache/tez/pull/345#issuecomment-2094565263 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 12m 34s | Docker mode

Re: [PR] TEZ-4552. Upgrade protobuf to 3.24.4. [tez]

2024-05-04 Thread via GitHub
slfan1989 commented on PR #345: URL: https://github.com/apache/tez/pull/345#issuecomment-2094555633 @abstractdog @Aggarwal-Raghav @BilwaST Thank you for paying attention to this pr! The reason I want to upgrade protobuf is because there are some CVE vulnerabilities in lower versions of

Re: [PR] TEZ-4552. Upgrade protobuf to 3.23.4. [tez]

2024-05-04 Thread via GitHub
BilwaST commented on PR #345: URL: https://github.com/apache/tez/pull/345#issuecomment-2094554985 Thanks for your patch @slfan1989. Looks good to me -- 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

Re: [PR] [TEZ-4558] Update build setup maven version and enforcer minimum to correct minimum [tez]

2024-05-03 Thread via GitHub
jteagles merged PR #350: URL: https://github.com/apache/tez/pull/350 -- 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 unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

Re: [PR] [TEZ-4558] Update build setup maven version and enforcer minimum to correct minimum [tez]

2024-05-03 Thread via GitHub
tez-yetus commented on PR #350: URL: https://github.com/apache/tez/pull/350#issuecomment-2093419155 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 33s | Docker mode

Re: [PR] TEZ-4557: Revert TEZ-4303, NoClassDefFoundError because of missing httpclient jar [tez]

2024-05-03 Thread via GitHub
Aggarwal-Raghav commented on PR #349: URL: https://github.com/apache/tez/pull/349#issuecomment-2093284646 @abstractdog, can you please review this? -- 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

Re: [PR] [TEZ-4558] Update build setup maven version and enforcer minimum to correct minimum [tez]

2024-05-03 Thread via GitHub
jteagles commented on PR #350: URL: https://github.com/apache/tez/pull/350#issuecomment-2093263201 @ayushtkn @zabetak , bumped minimum to 3.6.3. Rebuilt with an empty m2 cache to make sure all https dependencies will go through -- This is an automated message from the Apache Git

Re: [PR] [TEZ-4558] Update build setup maven version and enforcer minimum to correct minimum [tez]

2024-05-03 Thread via GitHub
ayushtkn commented on PR #350: URL: https://github.com/apache/tez/pull/350#issuecomment-2093187191 If everything works from 3.6.3 then it should be good -- 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

Re: [PR] [TEZ-4558] Update build setup maven version and enforcer minimum to correct minimum [tez]

2024-05-03 Thread via GitHub
jteagles commented on PR #350: URL: https://github.com/apache/tez/pull/350#issuecomment-2093158411 @zabetak , @ayushtkn Plugin API compatibility requires maven 3.6.3 or higher according to the official documentation https://maven.apache.org/docs/history.html How do we feel about

Re: [PR] [TEZ-4558] Update build setup maven version and enforcer minimum to correct minimum [tez]

2024-05-02 Thread via GitHub
tez-yetus commented on PR #350: URL: https://github.com/apache/tez/pull/350#issuecomment-2091103790 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 27m 7s | Docker mode

Re: [PR] TEZ-4557: Revert TEZ-4303, NoClassDefFoundError because of missing httpclient jar [tez]

2024-05-02 Thread via GitHub
tez-yetus commented on PR #349: URL: https://github.com/apache/tez/pull/349#issuecomment-2090887706 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 23m 22s | Docker mode

[PR] TEZ-4557: Revert TEZ-4303, NoClassDefFoundError because of missing httpclient jar [tez]

2024-05-02 Thread via GitHub
Aggarwal-Raghav opened a new pull request, #349: URL: https://github.com/apache/tez/pull/349 Please refer to [TEZ-4557](https://issues.apache.org/jira/browse/TEZ-4557), for error stacktrace. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] TEZ-4553. Default task scheduler to DagAwareTaskScheduler to avoid hang in TEZ-3535 [tez]

2024-05-01 Thread via GitHub
jteagles commented on PR #347: URL: https://github.com/apache/tez/pull/347#issuecomment-2088598837 Thanks, @ayushtkn. -- 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

  1   2   3   4   5   6   7   8   9   >