[MGNLACTIVATION-65] Remove dependency on EDU.oswego.cs.dl.util.concurrent Created: 06/Oct/09 Updated: 04/Nov/15 Resolved: 04/Nov/15 |
|
| Status: | Closed |
| Project: | Activation |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major |
| Reporter: | Jan Haderka | Assignee: | Ondrej Chytil |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoR: |
Empty
|
| Date of First Response: |
| Description |
|
Since we no longer support Java 1.4 this should be possible now. http://g.oswego.edu/dl/classes/EDU/oswego/cs/dl/util/concurrent/package-summary.html advises to migrate to the JDK5 java.util.concurrent package. The issue might need to be cloned for some other modules (rssaggregator, backup, ??) |
| Comments |
| Comment by Tobias Mattsson [ 06/Dec/10 ] |
|
Attached is a patch that replaces use of EDU.oswego with classes from JSR-166 java.util.concurrent. Use of EDU.oswego.cs.dl.util.concurrent.ClockDaemon in DelayedExecutor replaced with a ScheduledExecutor using a thread pool of only one thread to keep the same behavior. Use of EDU.oswego.cs.dl.util.concurrent.CountDown in BaseSyndicatorImpl and SimpleSyndicator replaced with CountDownLatch. Use of EDU.oswego.cs.dl.util.concurrent.PooledExecutor in ThreadPool replaced with ThreadPoolExecutor. Indentical behavior, i.e. 10 threads always on stand-by. License text in NOTICE.txt needs to be removed. |
| Comment by Magnolia International [ 06/Dec/10 ] |
|
NOTICE.txt is generated based on maven dependencies, so the notes should be removed automagically. |
| Comment by Jan Haderka [ 14/Mar/12 ] |
|
When tested we identified issues with activation (deadlocks) so once applied, activation should be again carefully tested. |
| Comment by Michael Mühlebach [ 04/Nov/15 ] |
|
Given the thousands of other issues we have open that are more highly requested, we won't be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes. |