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:
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
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
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
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
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
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
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
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
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
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
11 matches
Mail list logo