GitHub user simonati opened a pull request:
https://github.com/apache/flume/pull/141
FLUME-2752 Fix AvroSource startup resource leaks
Cleanup after Netty initialisation failes (call this.stop())
Make sure this.stop() releases the resources and end up the component in
a LifecycleAware.STOPPED state
Added junit tests to cover the invalid host and used port scenarios
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/simonati/flume FLUME-2752
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flume/pull/141.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #141
----
commit b22796fab4a9abd9edb85ae0d966ffac1785a1e5
Author: Attila Simon <s...@cloudera.com>
Date: 2017-06-29T06:21:44Z
FLUME-2752 Fix AvroSource startup resource leaks
Cleanup after Netty initialisation failes (call this.stop())
Make sure this.stop() releases the resources and end up the component in
a LifecycleAware.STOPPED state
Added junit test to cover the invalid host scenario
Added junit test to cover the used port scenario
Change-Id: Ia6309b597166fc4061864805fc4211c57f01c58e
commit 4b479013e21f77ef68da2a4f71462a7fda4ba932
Author: Attila Simon <s...@cloudera.com>
Date: 2017-06-29T07:13:15Z
Update rest of junit to pass
Change-Id: If9e4bf672359e7d5817cfc70438004b4358c9d10
commit b9ba593299ed87a0fe0db0183398a00409311546
Author: Attila Simon <s...@cloudera.com>
Date: 2017-06-29T07:20:40Z
fix indention
Change-Id: I37242fbb89f85ec8c31016d26c939bff82aee8f1
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
