sodonnel merged PR #6540:
URL: https://github.com/apache/hadoop/pull/6540
--
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:
hadoop-yetus commented on PR #6540:
URL: https://github.com/apache/hadoop/pull/6540#issuecomment-1936233134
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: |
sodonnel commented on PR #6540:
URL: https://github.com/apache/hadoop/pull/6540#issuecomment-1935695635
@sadanand48 The change LGTM, but there are two checkstyle warnings in the
test changes - could you fix them please?
--
This is an automated message from the Apache Git Service.
To
sadanand48 commented on PR #6540:
URL: https://github.com/apache/hadoop/pull/6540#issuecomment-1935358055
@sodonnel Could you please take a look?
--
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
hadoop-yetus commented on PR #6540:
URL: https://github.com/apache/hadoop/pull/6540#issuecomment-1934876888
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: |
sadanand48 opened a new pull request, #6540:
URL: https://github.com/apache/hadoop/pull/6540
### Description of PR
If the source file is EC, distcp without preserve option creates a 1
replication file (this is not intended).
This is because for an EC file getReplication() always