XComp commented on PR #23914:
URL: https://github.com/apache/flink/pull/23914#issuecomment-1853544386

   > The var tempFolder: Path = _ defines in scala is an object field already, 
for each @Test method, it is different.
   
   Thanks for clarification. My lack of Scala knowledge made me mix up the 
class and companion object declaration.
   
   > However, if the test of the same test is allowed to be executed 
concurrently, it may also cause path errors.
   
   That, I still don't get. Why would concurrent execution of tests cause this 
issue if each test class/instance has it's own temporary directory (because the 
tempFolder field is an object field)? Or do you mean if test methods are run 
concurrently? As far as I understand, that shouldn't happen: The concurrent 
test method execution has to be enabled explicitly using 
`@Execution(ExecutionMode.CONCURRENT)` (see [parent 
pom.xml:1745ff](https://github.com/apache/flink/blob/ea4cdc28651ad91defd4fc7b371a1f520ea7a262/pom.xml#L1745)).
 Or do I miss something here?  :thinking: 


-- 
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]

Reply via email to