[GitHub] [hadoop] snmvaughan commented on pull request #5343: HDFS-16905. Provide default hadoop.log.dir for tests

2023-02-06 Thread via GitHub
snmvaughan commented on PR #5343: URL: https://github.com/apache/hadoop/pull/5343#issuecomment-1419321373 The value set in the `hadoop-project/pom.xml` only applies to executions performed through Maven. Test execution through the IDE and alternative forms of test execution won't see this

[GitHub] [hadoop] snmvaughan commented on pull request #5343: HDFS-16905. Provide default hadoop.log.dir for tests

2023-02-03 Thread via GitHub
snmvaughan commented on PR #5343: URL: https://github.com/apache/hadoop/pull/5343#issuecomment-1416155276 I originally had it set to `./target/logs`, but changed it to `.` to match the other modules. I would think pointing it to `target` would make the most sense. Perhaps we should