[ 
https://issues.apache.org/jira/browse/HDFS-16905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17686033#comment-17686033
 ] 

ASF GitHub Bot commented on HDFS-16905:
---------------------------------------

virajjasani commented on PR #5343:
URL: https://github.com/apache/hadoop/pull/5343#issuecomment-1423053155

   > when i run tests in intellij idea I use its build (not maven) and set 
sysprops like -Dfs.s3a.scale.test.enabled=true for it. maybe that's needed here.
   
   I see, but since "scale" is already a profile, I actually enable the profile 
in IDEA so that even `-Dfs.s3a.scale.test.enabled=true` is not required, it 
will be done by profile anyways.
    
   <img width="280" alt="Screenshot 2023-02-08 at 10 17 00 AM" 
src="https://user-images.githubusercontent.com/34790606/217617224-afafad88-2e80-4d2d-96c4-1708f4fa940d.png";>
   
   
   > with this patch do hdfs tests log to the right place?
   
   This patch changes are actually not taken into effect by IDE because 
hadoop-project pom 
`<hadoop.log.dir>${project.build.directory}/log</hadoop.log.dir>` value is 
loaded by IDE as well.




> Provide default hadoop.log.dir for tests
> ----------------------------------------
>
>                 Key: HDFS-16905
>                 URL: https://issues.apache.org/jira/browse/HDFS-16905
>             Project: Hadoop HDFS
>          Issue Type: Test
>          Components: hdfs, hdfs-client
>    Affects Versions: 3.4.0, 3.3.5, 3.3.9
>         Environment: Tested using the Hadoop development environment Docker 
> image and an IDE on Mac
>            Reporter: Steve Vaughan
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 3.4.0, 3.3.9
>
>
> Provide a default directory configuration for test logging



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to