Github user ktzoumas commented on the pull request:
https://github.com/apache/flink/pull/606#issuecomment-93837926
This is one of the best project documentations that I have seen. Thanks a
lot Ufuk for doing this! I would merge this as is.
Few minor thoughts I had when skimming through the docs:
- Perhaps we can find a better font for the code samples (not sure if
applicable in this PR, perhaps this is a separate PR with a bunch of content
refactoring)
- Was it not possible to have the navigation on the left or was it a
conscious choice to have it on the right?
- Move Flink on Tez under Setup, since programming a job is the same,
deployment is different?
- "Programming guides" seems a bit too narrow for what it contains (e.g.,
Local/Cluster execution). Perhaps rename to "Usage"?
- Not sure if the whole internals section is better fit for the docs or the
wiki, but this is a ML discussion
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
