slfan1989 commented on PR #7019: URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2597346603
@Hexiaoqiao @steveloughran Thank you for approving this PR! If any issues arise in the future, I will continue to follow up. However, I remain optimistic since we haven't changed the core logic, and the primary changes are related to testing. I will also keep optimizing the code to prepare for a future upgrade to Jersey 3, and I already have some ideas for that. I plan to merge this PR in 2 working days. I will add the following members as co-authors in the commit to express my gratitude. @aajisaka @@virajjasani These two members provided the initial version of HADOOP-15984. @ayushtkn provided a customized version of yetus, which has made my overall testing process much easier. @pjfanning have submitted multiple PRs previously, effectively reducing the scope of changes for HADOOP-15984. @steveloughran and @Hexiaoqiao helped review the code and provided many valuable suggestions. I don't think I would have been able to push through without your encouragement. At the same time, I hope our JDK 17 upgrade goes smoothly. After this PR, we plan to submit a large-scale PR to upgrade from JUnit4 to JUnit5. We have already prepared for this locally. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
