[jira] [Commented] (BEAM-731) Replace DirectRunner with InProcessRunner

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

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

ASF GitHub Bot commented on BEAM-731:
-------------------------------------

GitHub user aaltay opened a pull request:

    https://github.com/apache/incubator-beam/pull/1240

    [BEAM-731] Replace DirectRunner with InProcessRunner

    Renames InprocessPipelineRunner to DirectPipelineRunner and removes the
    existing DirectPipelineRunner. Renamed the folder to direct to keep all
    related files in the same folder. Removed inprocess prefix from
    file names/class names/comments.
    
    DiskCachedPipelineRunner - not used, and it was a temporary solution
    to reduce the memory usage of DirectPipelineRunner. New
    DirectPipelineRunner does not have the same amount of memory usage.
    
    EagerPipelineRunner moved to direct folder and changed it to use new
    DirectPipelineRunner.

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

    $ git pull https://github.com/aaltay/incubator-beam inproc

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

    https://github.com/apache/incubator-beam/pull/1240.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 #1240
    
----
commit fd991280423e53bf9973fac1e9f21d87c0789cf9
Author: Ahmet Altay <al...@google.com>
Date:   2016-10-28T21:47:48Z

    Renames InprocessPipelineRunner to DirectPipelineRunner and removes the
    existing DirectPipelineRunner. Renamed the folder to direct to keep all
    related files in the same folder. Removed inprocess prefix from
    file names/class names/comments.
    
    DiskCachedPipelineRunner - not used, and it was a temporary solution
    to reduce the memory usage of DirectPipelineRunner. New
    DirectPipelineRunner does not have the same amount of memory usage.
    
    EagerPipelineRunner moved to direct folder and changed it to use new
    DirectPipelineRunner.

----


> Replace DirectRunner with InProcessRunner
> -----------------------------------------
>
>                 Key: BEAM-731
>                 URL: https://issues.apache.org/jira/browse/BEAM-731
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-py
>            Reporter: Ahmet Altay
>            Assignee: Ahmet Altay
>              Labels: sdk-consistency
>
> Remove the old DirectRunner and replace with the new InProcessRunner.
> There is an overhead for keeping both runners (testing/code maintenance etc.) InProcessRunner has been available for a while, it is tested enough for the being the default runner. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)