Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-29 Thread via GitHub
ayushtkn merged PR #6151: URL: https://github.com/apache/hadoop/pull/6151 -- 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:

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-29 Thread via GitHub
pjfanning commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1784221851 looks like the broken tests are already broken - see this unrelated PR but similar test failures there too -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-29 Thread via GitHub
pjfanning commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1784188272 There are some broken tests that need review. ``` hadoop.hdfs.server.datanode.TestDataNodeRollingUpgrade

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-29 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1784180744 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-28 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1783720241 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-27 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1783457560 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-27 Thread via GitHub
ayushtkn commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1783213744 Unless they have any user facing changes, I think we are safe, but if something change at the end user side, then maybe we can explore using the deprecated ones only -- This is an

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-27 Thread via GitHub
pjfanning commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1783116878 @ayushtkn I added https://github.com/apache/hadoop/pull/6151/commits/2561508eb4fb41b33343c80b87d11746efb17f7d but I'm wondering if in hadoop 3.3 that it might be better to use

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-26 Thread via GitHub
ayushtkn commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1782261141 Can you check the failures and javac warnings, on trunk everything was green.. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-21 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1773980040 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-14 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1763206788 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-13 Thread via GitHub
pjfanning commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1762298839 @steveloughran I have a backport PR for the module-info classes at https://github.com/apache/hadoop/pull/6188 -- This is an automated message from the Apache Git Service. To respond

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-13 Thread via GitHub
steveloughran commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1762048907 #6131 is in trunk now -- 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

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-13 Thread via GitHub
pjfanning commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1761218019 This is now blocked by https://issues.apache.org/jira/browse/HADOOP-18916 - commons-io 2.14.0 jar has module-info.class -- This is an automated message from the Apache Git Service. To

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-12 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1760565593 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-06 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1751574176 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-06 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1750506190 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-05 Thread via GitHub
hadoop-yetus commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1749850471 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-05 Thread via GitHub
ayushtkn commented on PR #6151: URL: https://github.com/apache/hadoop/pull/6151#issuecomment-1749620180 Blocked by: https://github.com/apache/hadoop/pull/6133#issuecomment-1749614219 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] HADOOP-18917. Upgrade to commons-io 2.14.0 (#6133) [hadoop]

2023-10-05 Thread via GitHub
pjfanning opened a new pull request, #6151: URL: https://github.com/apache/hadoop/pull/6151 backport #6133 -- 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,