You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by kennknowles <gi...@git.apache.org> on 2017/01/27 05:30:25 UTC
[GitHub] beam pull request #1858: [BEAM-362] Move state-oriented remnants of OldDoFn ...
GitHub user kennknowles opened a pull request:
https://github.com/apache/beam/pull/1858
[BEAM-362] Move state-oriented remnants of OldDoFn to runners/core-java
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:
- [x] Make sure the PR title is formatted like:
`[BEAM-<Jira issue #>] Description of pull request`
- [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
- [x] Replace `<Jira issue #>` in the title with the actual Jira issue
number, if there is one.
- [x] If this contribution is large, please file an Apache
[Individual Contributor License Agreement](https://www.apache.org/licenses/icla.txt).
---
This moves `WindowingInternals`, `StateInternals`, and `TimerInternals` out of the Java SDK and into runners/core-java. This is motivated by code health and user-facing API health but also easy first steps towards easing alterations to `WatermarkHoldState`'s relationship with `OutputTimeFn`, which is going away.
Kudos to @jkff for moving `OldDoFn` which made all of this trivial.
This will require a Dataflow worker image.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kennknowles/beam StateInternals
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/beam/pull/1858.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 #1858
----
commit bb6b544bcbd2e9e6be8cb1f2c69ae1bc992ac3d5
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:00:15Z
Eliminate use of InMemoryStateInternals in the Java SDK
commit 1fa7fc3d3969706a22d1a6969eefc45bb78a05b3
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:03:42Z
Move InMemoryStateInternals to runners/core-java
commit 7fbd347e67da62e79ec4a96b81a8c88a63796100
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:05:09Z
Delete long-deprecated StateContexts methods
commit 9f5aafb7be56f72ae8b9bcc69121830ea7e403ec
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:06:10Z
Move WindowingInternals to runners/core-java
commit 93dc4d588aecfd791d7304c3286b5e89baef77f5
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:09:41Z
Remove comment-only uses that block runners/core-java migrations
commit f33fb9a0bd98ccf4c3920becd5f50d68db525d36
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:13:14Z
Move StateInternalsFactory to runners/core-java
commit be048787ab798b702b0d4d8535496b36e42fcd13
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:13:42Z
Move TimerInternalsFactory to runners/core-java
commit cbe6ab56c937039675829b8cb1cf37d2c7ea4928
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:17:18Z
Move TimerInternals to runners/core-java
commit ffc28ca73fcf4f4a268ba75bc848037b2a8b3a7a
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:19:04Z
Move StateTable to runners/core-java
commit 9bac378b40f6c4e8d9b8856fef67a4e393f267d2
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:22:38Z
Move StateInternals to runners/core-java
commit 1cfc00b9a81c46de8b650b993ed2abb9c6dddf50
Author: Kenneth Knowles <kl...@google.com>
Date: 2017-01-27T05:26:04Z
Move StateNamespace and related to runners/core-java
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] beam pull request #1858: [BEAM-362] Move state-oriented remnants of OldDoFn ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/beam/pull/1858
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---