Skip to content

Conversation

@alpinegizmo
Copy link
Contributor

This is an update to the state intro for Flink 1.2, along with some introduction to the new state interfaces.

Copy link
Contributor

@aljoscha aljoscha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like these changes. 👍

One thing I would like to change is to change the ordering of keyed state and operator state because I think keyed state is the interface that users should mostly use while the operator state interfaces are for more obscure stuff.

of topic partitions and offsets as its Operator State.

#### Operator State
New interfaces in Flink 1.2 subsume the `Checkpointed` interface in Flink 1.0 and
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should document this without referring to old versions because that will quickly become outdated.

interface, or the `ListCheckpointed<T extends Serializable>` interface (which is semantically closer to the old
`Checkpointed` one).

[The Flink Function Migration documentation](../migration.html) has
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we shouldn't refer to the migration guide here because these interfaces are not only valid in a migration context but are valid for any Flink job.

@aljoscha
Copy link
Contributor

@alpinegizmo Could you please check out these changes: https://github.com/aljoscha/flink/tree/pr-3184-state-documentation? I took your PR, reshuffled the sections (keyed state is now mentioned before operator state) and added a section about operator state interfaces.

@aljoscha
Copy link
Contributor

aljoscha commented Jan 30, 2017

Thanks for your work, @alpinegizmo. 👍

Could you please close this PR? I merged it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants