Github user uce commented on the pull request:
https://github.com/apache/flink/pull/606#issuecomment-95117199
Addressed:
- Move Flink on Tez under Setup, since programming a job is the same,
deployment is different? => moved to setup
- The "beta" badge is not separated clearly enough from the rest of the
text => use badge and add to each beta page
- It would be cool to have the version somewhere prominent (navbar?) =>
Added to overview link
- I'm missing some space at the end of each page => OK
- Disqus
- Rebased and added Pyhton API links
Open:
- Not sure if the whole internals section is better fit for the docs or the
wiki, but this is a ML discussion => Let's start a discussion on dev@
- Was it not possible to have the navigation on the left or was it a
conscious choice to have it on the right? => It is possible, but it needed some
hacking and I didn't want to spend more time. Will change it soonish.
- "Programming guides" seems a bit too narrow for what it contains (e.g.,
Local/Cluster execution). Perhaps rename to "Usage"? => I didn't address this,
because we had it as programming guide before. I would also address this on the
ML.
---
Can someone check it out again and then merge.
---
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.
---
