[
https://issues.apache.org/jira/browse/IO-444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14000804#comment-14000804
]
Lukáš Petrovický edited comment on IO-444 at 5/17/14 3:27 PM:
--------------------------------------------------------------
Side note: I am using this new Tailer in my application
(https://github.com/triceo/splitlog) where I'm testing it well beyond Tailer's
own unit tests, and it's been performing well.
was (Author: triceo):
Side note: I am using this new Tailer in my application
(https://github.com/triceo/splitlog) where I'm testing it quite heavily, even
beyond Tailer's own unit tests, and it's been performing well.
> Convert Tailer to ScheduledExecutorService
> ------------------------------------------
>
> Key: IO-444
> URL: https://issues.apache.org/jira/browse/IO-444
> Project: Commons IO
> Issue Type: Improvement
> Components: Utilities
> Affects Versions: 2.5
> Reporter: Lukáš Petrovický
> Priority: Minor
>
> I have provided a pull request that converts Tailer to
> java.util.concurrent.ScheduledExecutorService:
> https://github.com/apache/commons-io/pull/2
> This has multiple benefits:
> - it provides periodic runs of the read operations without needing to invoke
> Thread.sleep().
> - it removes any looping from the run() method, making for cleaner code.
> - generally brings Tailer closer to what modern Java looks like.
> No public API changes have been made. All tests are passing without
> modifications. I kindly ask you to merge it.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
