[GitHub] [tez] rbalamohan commented on a diff in pull request #262: measured input stream

2023-01-16 Thread GitBox
rbalamohan commented on code in PR #262: URL: https://github.com/apache/tez/pull/262#discussion_r1071085336 ## tez-runtime-library/src/main/java/org/apache/tez/runtime/library/common/shuffle/Fetcher.java: ## @@ -563,7 +564,7 @@ private HostFetchResult

[GitHub] [tez] abstractdog commented on a diff in pull request #262: measured input stream

2023-01-16 Thread GitBox
abstractdog commented on code in PR #262: URL: https://github.com/apache/tez/pull/262#discussion_r1071047228 ## tez-runtime-library/src/main/java/org/apache/tez/runtime/library/common/shuffle/Fetcher.java: ## @@ -563,7 +564,7 @@ private HostFetchResult

[GitHub] [tez] abstractdog commented on a diff in pull request #262: measured input stream

2023-01-16 Thread GitBox
abstractdog commented on code in PR #262: URL: https://github.com/apache/tez/pull/262#discussion_r1071047228 ## tez-runtime-library/src/main/java/org/apache/tez/runtime/library/common/shuffle/Fetcher.java: ## @@ -563,7 +564,7 @@ private HostFetchResult

[GitHub] [tez] rbalamohan commented on a diff in pull request #262: measured input stream

2023-01-16 Thread GitBox
rbalamohan commented on code in PR #262: URL: https://github.com/apache/tez/pull/262#discussion_r1070957234 ## tez-runtime-library/src/main/java/org/apache/tez/runtime/library/common/shuffle/Fetcher.java: ## @@ -563,7 +564,7 @@ private HostFetchResult

[GitHub] [tez] tez-yetus commented on pull request #262: measured input stream

2023-01-12 Thread GitBox
tez-yetus commented on PR #262: URL: https://github.com/apache/tez/pull/262#issuecomment-1380547326 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 22s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #259: TEZ-4463: Upgrade jquery-ui to 1.13.0 due to CVEs

2023-01-10 Thread GitBox
tez-yetus commented on PR #259: URL: https://github.com/apache/tez/pull/259#issuecomment-1377366836 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 36m 52s | Docker mode

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2023-01-10 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1376875761 @steveloughran Not checked it yet. Will check and let you know -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [tez] steveloughran commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2023-01-09 Thread GitBox
steveloughran commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1376863511 Any reason in the logs for the cluster not coming up? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [tez] abstractdog merged pull request #258: TEZ-4462: update common-codec to 1.13 to fix vulnerability SNYK-JAVA-…

2023-01-05 Thread GitBox
abstractdog merged PR #258: URL: https://github.com/apache/tez/pull/258 -- 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:

[GitHub] [tez] difin commented on pull request #258: TEZ-4462: update common-codec to 1.13 to fix vulnerability SNYK-JAVA-…

2023-01-05 Thread GitBox
difin commented on PR #258: URL: https://github.com/apache/tez/pull/258#issuecomment-1372378101 Hi @rbalamohan, thanks a lot for the review! Could you please merge it? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2023-01-03 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1370544019 @abstractdog Upgrade to Hadoop 3.3.4 is failing tests in Tez as follows : ` [INFO] Running org.apache.tez.common.TestTezCommonUtils [ERROR] Tests run: 1, Failures: 0, Errors: 1,

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2023-01-03 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1370499506 @rbalamohan @jteagles Can you please approve this PR. Also @abstractdog are we going with the tez 0.10.3 release which will have hadoop 3.3.4. -- This is an automated message from the

[GitHub] [tez] rbalamohan commented on pull request #258: TEZ-4462: update common-codec to 1.13 to fix vulnerability SNYK-JAVA-…

2023-01-03 Thread GitBox
rbalamohan commented on PR #258: URL: https://github.com/apache/tez/pull/258#issuecomment-1370343622 LGTM. +1 -- 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

[GitHub] [tez] tez-yetus commented on pull request #261: Bump jettison from 1.5.1 to 1.5.2

2023-01-02 Thread GitBox
tez-yetus commented on PR #261: URL: https://github.com/apache/tez/pull/261#issuecomment-1369271733 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 36m 3s | Docker mode

[GitHub] [tez] dependabot[bot] closed pull request #244: Bump jettison from 1.3.4 to 1.5.1

2023-01-02 Thread GitBox
dependabot[bot] closed pull request #244: Bump jettison from 1.3.4 to 1.5.1 URL: https://github.com/apache/tez/pull/244 -- 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

[GitHub] [tez] dependabot[bot] commented on pull request #244: Bump jettison from 1.3.4 to 1.5.1

2023-01-02 Thread GitBox
dependabot[bot] commented on PR #244: URL: https://github.com/apache/tez/pull/244#issuecomment-1369244159 Superseded by #261. -- 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

[GitHub] [tez] dependabot[bot] opened a new pull request, #261: Bump jettison from 1.5.1 to 1.5.2

2023-01-02 Thread GitBox
dependabot[bot] opened a new pull request, #261: URL: https://github.com/apache/tez/pull/261 Bumps [jettison](https://github.com/jettison-json/jettison) from 1.5.1 to 1.5.2. Release notes Sourced from https://github.com/jettison-json/jettison/releases;>jettison's releases.

[GitHub] [tez] abstractdog merged pull request #260: TEZ-4464: Update some links to https

2023-01-02 Thread GitBox
abstractdog merged PR #260: URL: https://github.com/apache/tez/pull/260 -- 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:

[GitHub] [tez] tez-yetus commented on pull request #260: TEZ-4464: Update some links to https

2022-12-23 Thread GitBox
tez-yetus commented on PR #260: URL: https://github.com/apache/tez/pull/260#issuecomment-1363966393 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 39m 40s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #260: TEZ-4464: Update some links to https

2022-12-23 Thread GitBox
tez-yetus commented on PR #260: URL: https://github.com/apache/tez/pull/260#issuecomment-1363966328 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 40m 8s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #260: TEZ-4464: Update some links to https

2022-12-23 Thread GitBox
tez-yetus commented on PR #260: URL: https://github.com/apache/tez/pull/260#issuecomment-1363937785 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 30s | Docker mode

[GitHub] [tez] raboof commented on pull request #260: TEZ-4464: Update some links to https

2022-12-23 Thread GitBox
raboof commented on PR #260: URL: https://github.com/apache/tez/pull/260#issuecomment-1363925982 > Please justify why no new tests are needed for this patch. no tests were added because adding a link checker to the CI seems beyond the scope of this PR. > Also please list what

[GitHub] [tez] abstractdog commented on pull request #260: Update some links to https

2022-12-23 Thread GitBox
abstractdog commented on PR #260: URL: https://github.com/apache/tez/pull/260#issuecomment-1363921100 thanks for this contribution @raboof , can you please create jira also? https://issues.apache.org/jira/projects/TEZ/ -- This is an automated message from the Apache Git Service. To

[GitHub] [tez] tez-yetus commented on pull request #260: Update some links to https

2022-12-23 Thread GitBox
tez-yetus commented on PR #260: URL: https://github.com/apache/tez/pull/260#issuecomment-1363916885 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 31m 43s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #259: TEZ-4463: Upgrade jquery-ui to 1.13.0 due to CVEs

2022-12-13 Thread GitBox
tez-yetus commented on PR #259: URL: https://github.com/apache/tez/pull/259#issuecomment-1348484300 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 1s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #258: TEZ-4462: update common-codec to 1.13 to fix vulnerability SNYK-JAVA-…

2022-12-12 Thread GitBox
tez-yetus commented on PR #258: URL: https://github.com/apache/tez/pull/258#issuecomment-1347353645 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 31m 50s | Docker mode

[GitHub] [tez] difin opened a new pull request, #258: TEZ-4462: update common-codec to 1.13 to fix vulnerability SNYK-JAVA-…

2022-12-12 Thread GitBox
difin opened a new pull request, #258: URL: https://github.com/apache/tez/pull/258 …COMMONSCODEC-561518 -- 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,

[GitHub] [tez] abstractdog commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
abstractdog commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345477999 @jteagles : can you please review this? serious bug with shufflehandler, unit test added -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345404143 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 35m 23s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345404141 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 38m 16s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345403567 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 11s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345399634 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 26s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345399319 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 31s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345395029 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345394515 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 41s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-10 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1345190004 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 27s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #257: TEZ-4460: Read timed out in shuffle handler - incorrect usage of EMPTY_LAST_CONTENT and channel write

2022-12-09 Thread GitBox
tez-yetus commented on PR #257: URL: https://github.com/apache/tez/pull/257#issuecomment-1344593096 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 31m 58s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #256: TEZ-4394: Netty4 ShuffleHandler: should use 1 boss thread

2022-12-09 Thread GitBox
tez-yetus commented on PR #256: URL: https://github.com/apache/tez/pull/256#issuecomment-1344134981 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 31m 59s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #255: TEZ-4461: Upgrade spring-core to 5.3.20 in wro4j-maven-plugin

2022-12-07 Thread GitBox
tez-yetus commented on PR #255: URL: https://github.com/apache/tez/pull/255#issuecomment-1342041223 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #255: TEZ-4461: Upgrade spring-core to 5.3.20 in wro4j-maven-plugin

2022-12-07 Thread GitBox
tez-yetus commented on PR #255: URL: https://github.com/apache/tez/pull/255#issuecomment-1341004355 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 7s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #254: Bump express from 4.15.2 to 4.18.2 in /tez-ui/src/main/webapp

2022-12-06 Thread GitBox
tez-yetus commented on PR #254: URL: https://github.com/apache/tez/pull/254#issuecomment-1340172518 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 35m 56s | Docker mode

[GitHub] [tez] dependabot[bot] opened a new pull request, #254: Bump express from 4.15.2 to 4.18.2 in /tez-ui/src/main/webapp

2022-12-06 Thread GitBox
dependabot[bot] opened a new pull request, #254: URL: https://github.com/apache/tez/pull/254 Bumps [express](https://github.com/expressjs/express) from 4.15.2 to 4.18.2. Release notes Sourced from https://github.com/expressjs/express/releases;>express's releases. 4.18.2

[GitHub] [tez] guptashailesh92 commented on pull request #248: TEZ-4352: Add a web ui interface for TezChild

2022-12-05 Thread GitBox
guptashailesh92 commented on PR #248: URL: https://github.com/apache/tez/pull/248#issuecomment-1338899397 Sure, Thanks a lot. -- 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

[GitHub] [tez] mayankkunwar commented on a diff in pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-12-02 Thread GitBox
mayankkunwar commented on code in PR #253: URL: https://github.com/apache/tez/pull/253#discussion_r1037903486 ## pom.xml: ## @@ -758,7 +759,13 @@ org.bouncycastle bcprov-jdk15on -1.65 +${bouncycastle.version} +test + +

[GitHub] [tez] abstractdog merged pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-12-02 Thread GitBox
abstractdog merged PR #253: URL: https://github.com/apache/tez/pull/253 -- 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:

[GitHub] [tez] abstractdog commented on a diff in pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-12-02 Thread GitBox
abstractdog commented on code in PR #253: URL: https://github.com/apache/tez/pull/253#discussion_r1037902411 ## pom.xml: ## @@ -758,7 +759,13 @@ org.bouncycastle bcprov-jdk15on -1.65 +${bouncycastle.version} +test + +

[GitHub] [tez] mayankkunwar commented on a diff in pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-12-02 Thread GitBox
mayankkunwar commented on code in PR #253: URL: https://github.com/apache/tez/pull/253#discussion_r1037901361 ## pom.xml: ## @@ -758,7 +759,13 @@ org.bouncycastle bcprov-jdk15on -1.65 +${bouncycastle.version} +test + +

[GitHub] [tez] abstractdog commented on a diff in pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-12-02 Thread GitBox
abstractdog commented on code in PR #253: URL: https://github.com/apache/tez/pull/253#discussion_r1037898067 ## pom.xml: ## @@ -758,7 +759,13 @@ org.bouncycastle bcprov-jdk15on -1.65 +${bouncycastle.version} +test + +

[GitHub] [tez] mayankkunwar commented on pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-12-02 Thread GitBox
mayankkunwar commented on PR #253: URL: https://github.com/apache/tez/pull/253#issuecomment-1334884353 This is just a dependency upgrade, which should not require any additional tests. The changes are tested through existing unit tests. -- This is an automated message from the Apache Git

[GitHub] [tez] tez-yetus commented on pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-12-02 Thread GitBox
tez-yetus commented on PR #253: URL: https://github.com/apache/tez/pull/253#issuecomment-1334880863 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 20s | Docker mode

[GitHub] [tez] abstractdog commented on pull request #248: TEZ-4352: Add a web ui interface for TezChild

2022-12-01 Thread GitBox
abstractdog commented on PR #248: URL: https://github.com/apache/tez/pull/248#issuecomment-1333623812 hey, good questions @guptashailesh92 , here are my thoughts so far: 1. Consumer can be immediately a human being troubleshooting on a cluster, and also TezAM if we implement

[GitHub] [tez] guptashailesh92 commented on pull request #248: TEZ-4352: Add a web ui interface for TezChild

2022-11-29 Thread GitBox
guptashailesh92 commented on PR #248: URL: https://github.com/apache/tez/pull/248#issuecomment-1331691243 Hi @abstractdog , I just have few basic questions. I donot have full context on it so if you can point to some doc that would be great. 1. Who will be the consumer of the API?

[GitHub] [tez] tez-yetus commented on pull request #253: TEZ-4458:Upgrade Bouncy Castle to 1.70 due to high CVEs

2022-11-29 Thread GitBox
tez-yetus commented on PR #253: URL: https://github.com/apache/tez/pull/253#issuecomment-1330616742 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 36m 25s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #252: Bump dependency-check-maven from 1.3.6 to 3.2.0

2022-11-28 Thread GitBox
tez-yetus commented on PR #252: URL: https://github.com/apache/tez/pull/252#issuecomment-1329124758 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 41s | Docker mode

[GitHub] [tez] abstractdog commented on pull request #251: TEZ-4457. tez.task.launch.cmd-opts default value is not really used b…

2022-11-28 Thread GitBox
abstractdog commented on PR #251: URL: https://github.com/apache/tez/pull/251#issuecomment-1328917349 thanks for this patch @qijiale76 can you please add a unit test that validates this issue and the resolution? a recent commit on cmd opts might give you ideas on that:

[GitHub] [tez] dependabot[bot] opened a new pull request, #252: Bump dependency-check-maven from 1.3.6 to 3.2.0

2022-11-28 Thread GitBox
dependabot[bot] opened a new pull request, #252: URL: https://github.com/apache/tez/pull/252 Bumps [dependency-check-maven](https://github.com/jeremylong/DependencyCheck) from 1.3.6 to 3.2.0. Changelog Sourced from

[GitHub] [tez] abstractdog merged pull request #250: TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability

2022-11-28 Thread GitBox
abstractdog merged PR #250: URL: https://github.com/apache/tez/pull/250 -- 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:

[GitHub] [tez] abstractdog commented on pull request #250: TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability

2022-11-28 Thread GitBox
abstractdog commented on PR #250: URL: https://github.com/apache/tez/pull/250#issuecomment-1328906004 thanks for clarifying @amanraj2520 , +1 -- 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

[GitHub] [tez] tez-yetus commented on pull request #251: TEZ-4457. tez.task.launch.cmd-opts default value is not really used b…

2022-11-28 Thread GitBox
tez-yetus commented on PR #251: URL: https://github.com/apache/tez/pull/251#issuecomment-1328830008 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 27s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #251: TEZ-4457. tez.task.launch.cmd-opts default value is not really used b…

2022-11-28 Thread GitBox
tez-yetus commented on PR #251: URL: https://github.com/apache/tez/pull/251#issuecomment-1328717188 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 42s | Docker mode

[GitHub] [tez] qijiale76 commented on pull request #251: TEZ-4457. tez.task.launch.cmd-opts default value is not really used b…

2022-11-27 Thread GitBox
qijiale76 commented on PR #251: URL: https://github.com/apache/tez/pull/251#issuecomment-1328669788 The default value(jdk8) of `tez.task.launch.cmd-opts` is `-XX:+PrintGCDetails -verbose:gc -XX:+PrintGCTimeStamps -XX:+UseNUMA -XX:+UseParallelGC`. It is useful and should be really used by

[GitHub] [tez] qijiale76 opened a new pull request, #251: TEZ-4457. tez.task.launch.cmd-opts default value is not really used b…

2022-11-27 Thread GitBox
qijiale76 opened a new pull request, #251: URL: https://github.com/apache/tez/pull/251 …y default. -- 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:

[GitHub] [tez] amanraj2520 commented on pull request #250: TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability

2022-11-26 Thread GitBox
amanraj2520 commented on PR #250: URL: https://github.com/apache/tez/pull/250#issuecomment-1328039213 Hi @abstractdog these dependencies come up as you build the 2.20.3 version. They are required to build the is-valid-json 2.20.3. I have not made manual changes to this file. They are

[GitHub] [tez] tez-yetus commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-26 Thread GitBox
tez-yetus commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1328025607 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 36m 14s | Docker mode

[GitHub] [tez] abstractdog commented on pull request #187: TEZ-4387: use new doclet API in Java 9+

2022-11-26 Thread GitBox
abstractdog commented on PR #187: URL: https://github.com/apache/tez/pull/187#issuecomment-1328023308 okay thanks, let me take a look next week -- 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

[GitHub] [tez] abstractdog commented on a diff in pull request #250: TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability

2022-11-26 Thread GitBox
abstractdog commented on code in PR #250: URL: https://github.com/apache/tez/pull/250#discussion_r1032769819 ## tez-ui/src/main/webapp/yarn.lock: ## @@ -2672,12 +2672,17 @@ is-integer@^1.0.4: dependencies: is-finite "^1.0.0" -is-my-json-valid@^2.12.4: - version

[GitHub] [tez] abstractdog merged pull request #197: TEZ-4396 Ensure utility classes have only private (default) constructors

2022-11-26 Thread GitBox
abstractdog merged PR #197: URL: https://github.com/apache/tez/pull/197 -- 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:

[GitHub] [tez] abstractdog commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-25 Thread GitBox
abstractdog commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1327999403 I believe from tez->hadoop point of view, we usually rely only on precommit testing (including lots of minicluster tests) if something is broken from hive -> tez -> hadoop, we should

[GitHub] [tez] steveloughran commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-25 Thread GitBox
steveloughran commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1327384613 3.3.4 should be good; if not try with a 3.3.5 snapshot (build yourself) and see if that fixes things. i'd suggest going with 3.3.4 if it works, so the 3.3.5 release isn't a

[GitHub] [tez] amanraj2520 commented on pull request #250: TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability

2022-11-25 Thread GitBox
amanraj2520 commented on PR #250: URL: https://github.com/apache/tez/pull/250#issuecomment-1327166634 @abstractdog Can you please check and approve 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

[GitHub] [tez] tez-yetus commented on pull request #250: TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability

2022-11-24 Thread GitBox
tez-yetus commented on PR #250: URL: https://github.com/apache/tez/pull/250#issuecomment-1327079380 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 7s | Docker mode

[GitHub] [tez] amanraj2520 opened a new pull request, #250: TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability

2022-11-24 Thread GitBox
amanraj2520 opened a new pull request, #250: URL: https://github.com/apache/tez/pull/250 TEZ-4456: Upgraded is-my-json-valid version to 2.20.3 to fix the vulnerability. The parent JIRA can be tracked under : https://issues.apache.org/jira/browse/TEZ-4419 -- This is an automated

[GitHub] [tez] tez-yetus commented on pull request #197: TEZ-4396 Ensure utility classes have only private (default) constructors

2022-11-24 Thread GitBox
tez-yetus commented on PR #197: URL: https://github.com/apache/tez/pull/197#issuecomment-1326743888 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 32s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #197: TEZ-4396 Ensure utility classes have only private (default) constructors

2022-11-24 Thread GitBox
tez-yetus commented on PR #197: URL: https://github.com/apache/tez/pull/197#issuecomment-1326446261 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s | Docker mode

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-24 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1326349510 @abstractdog @steveloughran Can you please look into 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

[GitHub] [tez] abstractdog commented on pull request #248: TEZ-4352: Add a web ui interface for TezChild

2022-11-24 Thread GitBox
abstractdog commented on PR #248: URL: https://github.com/apache/tez/pull/248#issuecomment-1326298055 can I have a review on this @rbalamohan? the (optional) web ui interface of TezChild is supposed to be the starting point for further improvements like

[GitHub] [tez] abstractdog merged pull request #249: TEZ-4455: Add LoggingHandler in ShuffleHandler pipeline for better debuggability

2022-11-23 Thread GitBox
abstractdog merged PR #249: URL: https://github.com/apache/tez/pull/249 -- 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:

[GitHub] [tez] tez-yetus commented on pull request #197: TEZ-4396 Ensure utility classes have only private (default) constructors

2022-11-23 Thread GitBox
tez-yetus commented on PR #197: URL: https://github.com/apache/tez/pull/197#issuecomment-1325079651 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 28s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #249: TEZ-4455: Add LoggingHandler in ShuffleHandler pipeline for better de…

2022-11-22 Thread GitBox
tez-yetus commented on PR #249: URL: https://github.com/apache/tez/pull/249#issuecomment-1324213507 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 28s | Docker mode

[GitHub] [tez] difin opened a new pull request, #249: TEZ-4455: Add LoggingHandler in ShuffleHandler pipeline for better de…

2022-11-22 Thread GitBox
difin opened a new pull request, #249: URL: https://github.com/apache/tez/pull/249 …buggability. **What changes were proposed in this pull request?** Adding LoggingHandler in ShuffleHandler pipeline for better debuggability. **Why are the changes needed?** For corner case

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-22 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1324009932 @steveloughran So do you think having a new Tez release with 3.3.4 should help or not? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [tez] steveloughran commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-22 Thread GitBox
steveloughran commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1323968264 3.3.4 is the one where we fixed the tez incompatibilities for you  i plan to cut the 3.3.5 rc0 this week but it's a test of the rc process (x86 and arm) rather than something we

[GitHub] [tez] tez-yetus commented on pull request #197: TEZ-4396 Ensure utility classes have only private (default) constructors

2022-11-22 Thread GitBox
tez-yetus commented on PR #197: URL: https://github.com/apache/tez/pull/197#issuecomment-1323879701 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 52s | Docker mode

[GitHub] [tez] LA-Toth commented on pull request #187: TEZ-4387: use new doclet API in Java 9+

2022-11-22 Thread GitBox
LA-Toth commented on PR #187: URL: https://github.com/apache/tez/pull/187#issuecomment-1323360858 I had only one problem, the tests are too strict and cannot handle properly my solution. So I always get -1 for something. I don't think it's an error in my PR, I can't do anything. -- This

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-21 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-138247 Hi @ayushtkn I am familiar with this error. This happens because Tez 0.10.1 is using Hadoop 3.1.3 which in turn uses jetty 9.3* which does not have this method but when we upgrade

[GitHub] [tez] ayushtkn commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-21 Thread GitBox
ayushtkn commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1322201634 I was talking about https://github.com/apache/hive/pull/3279 here we tried till 3.3.3 and all the unit tests were passing but on actual cluster things weren't working. So, I meant apart

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-21 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1322109580 Hi @ayushtkn Thanks a lot for looking into this I checked the PR you were involved about upgrading Hive to 3.3.4 so I am referring this PR : https://github.com/apache/hive/pull/3578

[GitHub] [tez] ayushtkn commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-21 Thread GitBox
ayushtkn commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1322025347 Hi @amanraj2520 Just curious: What specifically you are targeting from Hadoop-3.3.4, mostly there would be some dependency upgrades in 3.3.4 & they must be in sync with the versions in

[GitHub] [tez] amanraj2520 commented on pull request #213: TEZ-4420: Upgrade to Hadoop 3.3.4

2022-11-21 Thread GitBox
amanraj2520 commented on PR #213: URL: https://github.com/apache/tez/pull/213#issuecomment-1321956832 Hi @ayushtkn @abstractdog @pjfanning @steveloughran, I am driving the hive-3.2.0 release in Open Source for which we have decided to upgrade to hadoop 3.3.4. But since Tez 0.10.2 is

[GitHub] [tez] tez-yetus commented on pull request #169: TEZ-4357: Report url to logs in case of fetcher connection failure

2022-11-17 Thread GitBox
tez-yetus commented on PR #169: URL: https://github.com/apache/tez/pull/169#issuecomment-1319653146 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 39s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #169: TEZ-4357: Report url to logs in case of fetcher connection failure

2022-11-17 Thread GitBox
tez-yetus commented on PR #169: URL: https://github.com/apache/tez/pull/169#issuecomment-1318485839 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 32m 41s | Docker mode

[GitHub] [tez] tez-yetus commented on pull request #169: TEZ-4357: Report url to logs in case of fetcher connection failure

2022-11-17 Thread GitBox
tez-yetus commented on PR #169: URL: https://github.com/apache/tez/pull/169#issuecomment-1318449830 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 10s | Docker mode

[GitHub] [tez] abstractdog closed pull request #94: TEZ-4274: Log Credential Tokens

2022-11-17 Thread GitBox
abstractdog closed pull request #94: TEZ-4274: Log Credential Tokens URL: https://github.com/apache/tez/pull/94 -- 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,

[GitHub] [tez] abstractdog commented on pull request #94: TEZ-4274: Log Credential Tokens

2022-11-17 Thread GitBox
abstractdog commented on PR #94: URL: https://github.com/apache/tez/pull/94#issuecomment-1318436723 closing this as housekeeping, please reopen if needed -- 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

[GitHub] [tez] dependabot[bot] commented on pull request #237: Bump hadoop-common from 3.3.1 to 3.3.3

2022-11-17 Thread GitBox
dependabot[bot] commented on PR #237: URL: https://github.com/apache/tez/pull/237#issuecomment-1318433733 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me

[GitHub] [tez] abstractdog commented on pull request #237: Bump hadoop-common from 3.3.1 to 3.3.3

2022-11-17 Thread GitBox
abstractdog commented on PR #237: URL: https://github.com/apache/tez/pull/237#issuecomment-1318433688 we're about to do this in TEZ-4420, closing 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

[GitHub] [tez] abstractdog closed pull request #237: Bump hadoop-common from 3.3.1 to 3.3.3

2022-11-17 Thread GitBox
abstractdog closed pull request #237: Bump hadoop-common from 3.3.1 to 3.3.3 URL: https://github.com/apache/tez/pull/237 -- 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

[GitHub] [tez] abstractdog closed pull request #136: we have installed hadoop 3.2.X and hive 3.1.2 and tez installed 0.10

2022-11-17 Thread GitBox
abstractdog closed pull request #136: we have installed hadoop 3.2.X and hive 3.1.2 and tez installed 0.10 URL: https://github.com/apache/tez/pull/136 -- 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

[GitHub] [tez] abstractdog commented on pull request #136: we have installed hadoop 3.2.X and hive 3.1.2 and tez installed 0.10

2022-11-17 Thread GitBox
abstractdog commented on PR #136: URL: https://github.com/apache/tez/pull/136#issuecomment-1318428007 closing this PR as it seems to have lots of noise and there is no Jira ticket mentioned please reopen if you're about to work on this @abinanths recommended hive+tez combinations

[GitHub] [tez] abstractdog closed pull request #121: Branch 0.10.0

2022-11-17 Thread GitBox
abstractdog closed pull request #121: Branch 0.10.0 URL: https://github.com/apache/tez/pull/121 -- 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:

  1   2   3   4   5   6   7   8   9   10   >