You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Vaclav Plajt (JIRA)" <ji...@apache.org> on 2018/06/18 08:13:00 UTC

[jira] [Commented] (BEAM-4428) General Cleanup

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

Vaclav Plajt commented on BEAM-4428:
------------------------------------

Pull request: https://github.com/seznam/beam/pull/11

> General Cleanup
> ---------------
>
>                 Key: BEAM-4428
>                 URL: https://issues.apache.org/jira/browse/BEAM-4428
>             Project: Beam
>          Issue Type: Sub-task
>          Components: dsl-euphoria
>            Reporter: Vaclav Plajt
>            Assignee: Vaclav Plajt
>            Priority: Major
>             Fix For: Not applicable
>
>
> Address following items:
>  * package `org.apache.beam.sdk.extensions.euphoria.beam` deserves a better name. No need to have `beam` twice.
>  * Module `euphoria-local`should be removed.
>  * `README.md` is outdated
>  * `BeamExecutorContext` executor concept was removed and the `BeamExecutorContext` has become more something like `TranslationContext`. Rename It.
>  * `[BeamFlow.java |https://github.com/seznam/beam/pull/10/files/3499b45428aea26203d5c01e0cd1fc1dd62600f3#diff-ef45f0356073bc369bd471d2847b18d7]` is created by `create(...)` set of methods, that is not consistent with rest of the API rename it to 'of(...)'.
>  



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