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

2024-02-23 Thread via GitHub
tez-yetus commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-1961045473 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 24m 13s | 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-02-23 Thread via GitHub
zhengchenyu commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-1961012893 @abstractdog @rbalamohan 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

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

2024-02-23 Thread via GitHub
tez-yetus commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-1960986435 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | 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-02-23 Thread via GitHub
tez-yetus commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-1960951889 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | 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-02-23 Thread via GitHub
zhengchenyu closed pull request #336: TEZ-4542: Tez application may fail due to int overflow when record size is large and sort memory is low. 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

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

2024-02-23 Thread via GitHub
tez-yetus commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-1960916001 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | 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-02-23 Thread via GitHub
zhengchenyu commented on PR #336: URL: https://github.com/apache/tez/pull/336#issuecomment-1960893239 @abstractdog @rbalamohan 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