{"action":"created","issue":{"url":"https://api.github.com/repos/seznam/beam/issues/24","repository_url":"https://api.github.com/repos/seznam/beam","labels_url":"https://api.github.com/repos/seznam/beam/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/seznam/beam/issues/24/comments","events_url":"https://api.github.com/repos/seznam/beam/issues/24/events","html_url":"https://github.com/seznam/beam/pull/24","id":353526778,"node_id":"MDExOlB1bGxSZXF1ZXN0MjEwNTU2MTE0","number":24,"title":"[BEAM-3900] new translation","user":{"login":"dmvk","id":299781,"node_id":"MDQ6VXNlcjI5OTc4MQ==","avatar_url":"https://avatars1.githubusercontent.com/u/299781?v=4","gravatar_id":"","url":"https://api.github.com/users/dmvk","html_url":"https://github.com/dmvk","followers_url":"https://api.github.com/users/dmvk/followers","following_url":"https://api.github.com/users/dmvk/following{/other_user}","gists_url":"https://api.github.com/users/dmvk/gists{/gist_id}","starred_url":"https://api.github.com/users/dmvk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dmvk/subscriptions","organizations_url":"https://api.github.com/users/dmvk/orgs","repos_url":"https://api.github.com/users/dmvk/repos","events_url":"https://api.github.com/users/dmvk/events{/privacy}","received_events_url":"https://api.github.com/users/dmvk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-08-23T19:46:09Z","updated_at":"2018-09-18T07:51:23Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/seznam/beam/pulls/24","html_url":"https://github.com/seznam/beam/pull/24","diff_url":"https://github.com/seznam/beam/pull/24.diff","patch_url":"https://github.com/seznam/beam/pull/24.patch"},"body":"**Please** add a meaningful description for your change here\r\n\r\n------------------------\r\n\r\nFollow this checklist to help us incorporate your contribution quickly and easily:\r\n\r\n - [ ] Format the pull request title like `[BEAM-XXX] Fixes bug in ApproximateQuantiles`, where you replace `BEAM-XXX` with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.\r\n - [ ] If this contribution is large, please file an Apache [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).\r\n\r\nIt will help us expedite review of your Pull Request if you tag someone (e.g. `@username`) to look at it.\r\n\r\nPost-Commit Tests Status (on master branch)\r\n------------------------------------------------------------------------------------------------\r\n\r\nLang | SDK | Apex | Dataflow | Flink | Gearpump | Samza | Spark\r\n--- | --- | --- | --- | --- | --- | --- | ---\r\nGo | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Go_GradleBuild/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Go_GradleBuild/lastCompletedBuild/) | --- | --- | --- | --- | --- | ---\r\nJava | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_GradleBuild/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_GradleBuild/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark_Gradle/lastCompletedBuild/)\r\nPython | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Python_Verify/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python_Verify/lastCompletedBuild/) | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/) </br> [![Build Status](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/) | --- | --- | --- | ---\r\n\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/seznam/beam/issues/comments/422291886","html_url":"https://github.com/seznam/beam/pull/24#issuecomment-422291886","issue_url":"https://api.github.com/repos/seznam/beam/issues/24","id":422291886,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMjI5MTg4Ng==","user":{"login":"je-ik","id":14139263,"node_id":"MDQ6VXNlcjE0MTM5MjYz","avatar_url":"https://avatars1.githubusercontent.com/u/14139263?v=4","gravatar_id":"","url":"https://api.github.com/users/je-ik","html_url":"https://github.com/je-ik","followers_url":"https://api.github.com/users/je-ik/followers","following_url":"https://api.github.com/users/je-ik/following{/other_user}","gists_url":"https://api.github.com/users/je-ik/gists{/gist_id}","starred_url":"https://api.github.com/users/je-ik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/je-ik/subscriptions","organizations_url":"https://api.github.com/users/je-ik/orgs","repos_url":"https://api.github.com/users/je-ik/repos","events_url":"https://api.github.com/users/je-ik/events{/privacy}","received_events_url":"https://api.github.com/users/je-ik/received_events","type":"User","site_admin":false},"created_at":"2018-09-18T07:51:23Z","updated_at":"2018-09-18T07:51:23Z","author_association":"COLLABORATOR","body":"Just a few notes from the history of EUphoria, which could, but might not be relevant any more. That's up to you to decide.\r\n> Why not to have API to construct and set windowing only?\r\n\r\nThe original intent of Euphoria API was to use semantically relevant operators only - that is, let (or force in some sense) user to specify exactly *what* he wants accomplish and not *how* he wants to do it. This has some philosophical implications - one of them is that each operator (transformation) should provide some processing logic and deliver added value of its own. Operator Window as is in my opinion violates this, because there is no actual added value in it - it can be imagined just as coloring of input elements for further processing, so the Window itself is definitely not user's intent. That's why we decided to put it on stateful operators - because it is only there where it makes any semantic sense.\r\n>     * `Split` is somewhat hidden composite operator we support. It seems to me, that is should not be composite. It can be directl translated into Beam's [Partition](https://beam.apache.org/documentation/programming-guide/#partition) transform. Making it first class citizen would be also appropriate.\r\n\r\nOriginal design decision (might not be valid any more) was to support only single output Dataset - additional Dataset might be added as side outputs, but Split as defined didn't fit into this model, so it was added as \"facade\" around multiple operators (and yes, effectively making it hidden composite operator). The question here is, can we now fully support operators with multiple output Datasets? What are the consequences?\r\n\r\n>     * Sometimes Beam complains about unstable names or even naming collision. Since we do not enforce naming of operators, we may add unique generated names when used chooses not to give a name. Current state is that name of operator is assigned to each unnamed operator instance. That leads to the collisions.\r\nI think it is generally error not to name operators. The error is slightly higher if we automatically name them in a conflicting manner, but solving this actually doesn't solve too much for the user. The user might be forced to name the operators (at least in streaming mode).\r\n"}}
{"actor":{"display_login":"je-ik"}}