GitHub user StephanEwen opened a pull request:
https://github.com/apache/flink/pull/1570
[FLINK-3303] Move all non-batch specific classes in "flink-java" to
"flink-core"
Currently, `flink-java` has a lot of classes that are also needed by the
streaming API. For some future changes, some of them need to be used by classes
in `flink-core`. In particular, certain improvements to the state API are
blocked by the fact that certain classes are not in `flink-core`.
This pull request moves classes from `flink-java` to `flink-core` after the
following pattern:
- `flink-core` will contain all classes that are common across the batch
and streaming API.
- `flink-java` will contain all batch API specific classes (we may
eventually even think about renaming it to "flink-batch-java").
Because `flink-java` references `flink-core`, the relocation part is not a
breaking change.
### Breaking parts
This pull request contains [FLINK-3049], relocating the `Either` type to
`org.apache.flink.types`, from `org.apache.flink.api.java.typeutils`.
### Changes besides moving files to another project
- The `Keys` class is now part of `flink-core`. Utility behavior to
inject key-extracting mappers was pulled out into a separate utility class. The
class was also moved to `o.a.f.api.common.operators` from
`o.a.f.api.java.operators`. Since that class is not public API, this change is
not API breaking.
- Lots of tests are now also part of `flink-core`. Minor package
relocations on the tests (naturally not breaking).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/StephanEwen/incubator-flink move_to_core
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/1570.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 #1570
----
commit 2eccea20421cc22d24f219703aabc3d4da0ee1ac
Author: Stephan Ewen <[email protected]>
Date: 2016-01-31T22:08:18Z
[FLINK-3303] [core] Move Tuple classes to flink-core
commit 2b756aa2a819e4f65f8c45a0b85258129be70b2f
Author: Stephan Ewen <[email protected]>
Date: 2016-01-31T22:27:36Z
[FLINK-3049] [api breaking] Move 'Either' type to 'flink-core /
org.apache.flink.types'
commit 334d83204b933023728939d038478b66c1065280
Author: Stephan Ewen <[email protected]>
Date: 2016-01-31T22:28:32Z
[FLINK-3303] [core] Move all type utilities to flink-core
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
