[jira] [Commented] (FLINK-8812) Possible resource leak in Flip6

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/FLINK-8812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16403555#comment-16403555 ] 

ASF GitHub Bot commented on FLINK-8812:
---------------------------------------

GitHub user tillrohrmann opened a pull request:

    https://github.com/apache/flink/pull/5713

    [FLINK-8812] [flip6] Set managed memory for TaskExecutor to 80 MB in MiniCluster

    ## What is the purpose of the change
    
    In order to avoid problems with OOM exceptions, this commit sets the managed
    memory to 80 MB for TaskExecutors started by the MiniCluster.
    
    ## Does this pull request potentially affect one of the following parts:
    
      - Dependencies (does it add or upgrade a dependency): (no)
      - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: (no)
      - The serializers: (no)
      - The runtime per-record code paths (performance sensitive): (no)
      - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
      - The S3 file system connector: (no)
    
    ## Documentation
    
      - Does this pull request introduce a new feature? (no)
      - If yes, how is the feature documented? (not applicable)


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tillrohrmann/flink findResourceLeak

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/5713.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 #5713
    
----
commit 47f3f515ea75db20528079fef289de5120ae95c5
Author: Till Rohrmann <tr...@...>
Date:   2018-03-16T18:16:49Z

    [FLINK-9016] [flip6] Properly unregister jobs from JobMetricGroup
    
    This commit properly removes jobs from the JobMetricGroup once a job
    has reached a terminal state.

commit e42749cea4a0d35850c9482c8b31274aa081dbfa
Author: Till Rohrmann <tr...@...>
Date:   2018-03-17T16:53:35Z

    [FLINK-8812] [flip6] Set managed memory for TaskExecutor to 80 MB in MiniCluster
    
    In order to avoid problems with OOM exceptions, this commit sets the managed
    memory to 80 MB for TaskExecutors started by the MiniCluster.

----


> Possible resource leak in Flip6
> -------------------------------
>
>                 Key: FLINK-8812
>                 URL: https://issues.apache.org/jira/browse/FLINK-8812
>             Project: Flink
>          Issue Type: Bug
>          Components: Tests
>            Reporter: Chesnay Schepler
>            Assignee: Till Rohrmann
>            Priority: Blocker
>              Labels: flip-6
>             Fix For: 1.5.0
>
>
> In this build (https://travis-ci.org/zentol/flink/builds/347373839) I set the codebase to flip6 for half the profiles to find failing tests.
> The "libraries" job (https://travis-ci.org/zentol/flink/jobs/347373851) failed with an OutOfMemoryError.
> This could mean that there is a memory-leak somewhere.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)