Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-29 Thread via GitHub
hadoop-yetus commented on PR #6745: URL: https://github.com/apache/hadoop/pull/6745#issuecomment-2082067434 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| _ Prechecks

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-23 Thread via GitHub
hadoop-yetus commented on PR #6745: URL: https://github.com/apache/hadoop/pull/6745#issuecomment-2072123314 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| _ Prechecks

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-19 Thread via GitHub
hadoop-yetus commented on PR #6745: URL: https://github.com/apache/hadoop/pull/6745#issuecomment-2066719257 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-19 Thread via GitHub
kokonguyen191 commented on code in PR #6745: URL: https://github.com/apache/hadoop/pull/6745#discussion_r1571957646 ## hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DebugAdmin.java: ## @@ -641,6 +663,207 @@ private void closeBlockReaders() { }

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-18 Thread via GitHub
ZanderXu commented on code in PR #6745: URL: https://github.com/apache/hadoop/pull/6745#discussion_r1571614915 ## hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DebugAdmin.java: ## @@ -641,6 +663,207 @@ private void closeBlockReaders() { } +

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-18 Thread via GitHub
hadoop-yetus commented on PR #6745: URL: https://github.com/apache/hadoop/pull/6745#issuecomment-2064628230 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-18 Thread via GitHub
kokonguyen191 commented on PR #6745: URL: https://github.com/apache/hadoop/pull/6745#issuecomment-2063531719 Failed unit tests are unrelated and all pass locally. Hi @ayushtkn @ZanderXu , can help take a look if you are free, thanks! -- This is an automated message from the Apache

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-18 Thread via GitHub
hadoop-yetus commented on PR #6745: URL: https://github.com/apache/hadoop/pull/6745#issuecomment-2063491398 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-17 Thread via GitHub
hadoop-yetus commented on PR #6745: URL: https://github.com/apache/hadoop/pull/6745#issuecomment-2061806363 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

[PR] HDFS-17475. Add verifyReadable command to check if files are readable [hadoop]

2024-04-17 Thread via GitHub
kokonguyen191 opened a new pull request, #6745: URL: https://github.com/apache/hadoop/pull/6745 ### Description of PR Sometimes a job can fail due to one unreadable file down the line due to missing replicas or dead DNs or other reason. This command allows users to check whether