You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/05/23 11:06:32 UTC

[GitHub] [flink-web] curcur commented on a diff in pull request #540: Add Changelog State Backend blog post

curcur commented on code in PR #540:
URL: https://github.com/apache/flink-web/pull/540#discussion_r879324188


##########
_posts/2022-05-20-changelog-state-backend.md:
##########
@@ -338,6 +336,10 @@ We encourage you to try out this feature and assess the pros and cons of using i
 
 Please see the full documentation [here](https://nightlies.apache.org/flink/flink-docs-master/docs/ops/state/state_backends/#enabling-changelog).
 
+# Acknowledgments
+
+We thank Stephan Ewen and all the engineers who contributed to the project.

Review Comment:
   LOL, maybe add a bit more names and descriptions of what are they thanked for ...
   
   Otherwise, it is now reading like "we do not want to thank you"
   
   For example,
   We thank Stephan Ewen for the initial idea of the changelog, and many other engineers Piotr, Yu and Yun for design discussion and code review.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org