[
https://issues.apache.org/jira/browse/BEAM-4428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Vaclav Plajt updated BEAM-4428:
-------------------------------
Description:
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(...)'.
was:
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(...)'.
* Almost all Euphoria operators allows for windowing. But only some Beam
transformations are using it. We should remove windowing from Euphora operators
which are translated to Beam transformations which do not supports windowing.
Plus in beam windowing can be defined in Pipeline before the transformation
which uses it! Euphoria does not follow this principle.
> 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)
