ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jaikiran <...@git.apache.org>
Subject [GitHub] ant issue #81: Fix rare ConcurrentModificationException when running with Pa...
Date Wed, 12 Dec 2018 01:58:19 GMT
Github user jaikiran commented on the issue:

    https://github.com/apache/ant/pull/81
  
    > @jaikiran I think the test does exhibit the multithreaded scenario, I ran it prior
to the Project changes to ensure it did fail. My Java is a bit rusty, but I believe the 2
separate ExecutorService's created run concurrently.
    
    You are right indeed. When I replied previously, I did not focus on the second executor
submit and the fact that this task would run in parallel with the previous for loop that (might)
still be in progress.
    
    > My only concern is it's possible it may not catch the issue in the future as a regression
test due to the non-determinism, as well as the somewhat arbitrary loop count and extended
time taken to run the test (<100 milliseconds currently, but overall these types of tests
tend to add up).
    
    Agreed. So it's fine to not include the test.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Mime
View raw message