New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FLINK-21369][docs] Document Checkpoint Storage #14932
Conversation
Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community Automated ChecksLast check on commit 95e05ce (Fri Feb 12 19:46:25 UTC 2021) ✅no warnings Mention the bot in a comment to re-run the automated checks. Review Progress
Please see the Pull Request Review Guide for a full explanation of the review process. The Bot is tracking the review progress through labels. Labels are applied according to the order of the review items. For consensus, approval by a Flink committer of PMC member is required Bot commandsThe @flinkbot bot supports the following commands:
|
95e05ce
to
a0ff0d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically looks good, but I found a bunch of nits and had a question or two whether something is correct.
docs/content/docs/dev/datastream/fault-tolerance/checkpointing.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/fault-tolerance/checkpointing.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/fault-tolerance/checkpointing.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/fault-tolerance/schema_evolution.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/fault-tolerance/schema_evolution.md
Outdated
Show resolved
Hide resolved
4b952f2
to
d42451f
Compare
d42451f
to
030c2d5
Compare
@alpinegizmo thank you for the review, I hope I've addressed all your comments. Sorry for force pushing, I messed up a rebase. |
mergind |
What is the purpose of the change
Add documentation for the new Checkpoint Storage abstraction and new State Backend apis (HashMap and EmbeddedRocksDB).
I also cleaned up a few broken links and other misc I found along the way