[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-04-26 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1523661404 Ya, I thought like that. Thank you, @steveloughran and @ayushtkn . -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-04-28 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1527745355 Thank you, @ayushtkn . -- 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 co

[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-05-02 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1531688805 BTW, the current CI seems to complain for something. Let me check more while I'm here. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-06-03 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1575417532 Sorry for the delay, @ayushtkn . I converted it back. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-06-06 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1579808345 I updated the code and PR description by removing `Use outputDirectory instead of destDir` changes. -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-06-07 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1581572452 The ASF license check pipeline fails, but it looks irrelevant. ![Screenshot 2023-06-07 at 3 07 23 PM](https://github.com/apache/hadoop/assets/9700541/2f66f9f0-3a24-49e7-8a6d-63

[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-06-08 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1583141301 Thank you, @GauthamBanasandra . -- 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 sp

[GitHub] [hadoop] dongjoon-hyun commented on pull request #5592: HADOOP-18718. Fix several maven build warnings

2023-06-11 Thread via GitHub
dongjoon-hyun commented on PR #5592: URL: https://github.com/apache/hadoop/pull/5592#issuecomment-1586315895 Thank you so much, @ayushtkn , @steveloughran , @GauthamBanasandra . -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub