> On July 22, 2016, 12:37 a.m., Jake Maes wrote: > > docs/learn/documentation/versioned/index.html, lines 48-49 > > <https://reviews.apache.org/r/50317/diff/1/?file=1450706#file1450706line48> > > > > I'm not sure these should be nested in the Overview page. They are > > critical parts of the API and we want users to be able to find them easily.
Thanks for the comments. These two are on the main list when we go to https://samza.apache.org/ (left column list including "Documentation" which is for this doc, and "Configuration", "Javadocs", and etc.), so users can find them as easily as "Hello Samza", "FAQ". We may not want to list "Configuration" and "Javadocs" again in the table for https://samza.apache.org/learn/documentation/0.10/ as another individual sections, and also, all sections in https://samza.apache.org/learn/documentation/0.10/ are following the same format except "Configuration" and "Javadocs". - Fred ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50317/#review143165 ----------------------------------------------------------- On July 22, 2016, 12:26 a.m., Fred Ji wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50317/ > ----------------------------------------------------------- > > (Updated July 22, 2016, 12:26 a.m.) > > > Review request for samza, Jon Bringhurst, Jagadish Venkatraman, and Yi Pan > (Data Infrastructure). > > > Repository: samza > > > Description > ------- > > 1. resolve the inconsistent next section link at the end of each section on > samza.apache.org/learn/documentation/0.10/; > 2. resolove some broken links such as Coda Hale's metrics library and Linux > CGroup; > 3. update some sentences or rephrase to make description clearer. > > > Diffs > ----- > > docs/learn/documentation/versioned/api/overview.md > 6712344e84e19883b857e00549db2acb101c7e0e > docs/learn/documentation/versioned/comparisons/mupd8.md > 53417f91019c08d470d101685b6c40dad112bb18 > docs/learn/documentation/versioned/comparisons/storm.md > 58cb50895e7a419a7bede8911cd5b7f21fb487c4 > docs/learn/documentation/versioned/container/coordinator-stream.md > 98c1a845cda5ca2a53ee58b55508fc93492b982d > docs/learn/documentation/versioned/container/event-loop.md > 116238312df7071747cbbc14bc9c46f558755195 > docs/learn/documentation/versioned/container/metrics.md > 0423155537b0095b9efcfb40757e8de17b640ffd > docs/learn/documentation/versioned/container/samza-container.md > 55bec9813ca006757db046e6e85259b4d2e8c770 > docs/learn/documentation/versioned/container/state-management.md > 320392cd4412b459a093ab9657b7264cee42958b > docs/learn/documentation/versioned/container/windowing.md > b10e5d4eba61cb5da026af828d89d558457a6b16 > docs/learn/documentation/versioned/hdfs/producer.md > 0865a351b48c0b00e32282d2d7e44dfeae09b4ec > docs/learn/documentation/versioned/index.html > 1e79bd6f39ee67b5a2b74222d647342990bbbe23 > docs/learn/documentation/versioned/yarn/isolation.md > a76f6fd0f93482c085b15bb1b2c313a913508bcc > docs/learn/documentation/versioned/yarn/yarn-host-affinity.md > af6367da4f5611e208c9fb8426554c4a9c4ad6c8 > > Diff: https://reviews.apache.org/r/50317/diff/ > > > Testing > ------- > > Basic unit tests and checkstyleTest passed. > 1. Run unit tests: > ./gradlew clean build > 2. Check if your code follows the coding conventions: > ./gradlew checkstyleMain checkstyleTest > > > Thanks, > > Fred Ji > >