You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by simonati <gi...@git.apache.org> on 2017/06/29 08:01:17 UTC

[GitHub] flume pull request #141: FLUME-2752 Fix AvroSource startup resource leaks

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 <sa...@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 <sa...@cloudera.com>
Date:   2017-06-29T07:13:15Z

    Update rest of junit to pass
    
    Change-Id: If9e4bf672359e7d5817cfc70438004b4358c9d10

commit b9ba593299ed87a0fe0db0183398a00409311546
Author: Attila Simon <sa...@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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flume pull request #141: FLUME-2752 Fix AvroSource startup resource leaks

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/flume/pull/141


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---