[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

[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

[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

[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

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

2023-06-04 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

[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,

[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

[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