[ 
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)

Reply via email to