search

confluentinc / camus

Mirror of Linkedin's Camus
Apache License 2.0
0 stars 62 forks source link

Doc #6

Closed nehanarkhede closed 9 years ago

nehanarkhede commented 9 years ago

@Ishiihara Creating a PR for the docs so I can review them.

Ishiihara commented 9 years ago

@ewencp @nehanarkhede Addressed most comments. Ready for the second round of review.

ewencp commented 9 years ago

@Ishiihara Much improved. All my notes were minor tweaks or we have other issues filed to resolve them after this is merged, so +1 for me after those minor fixes.

Ishiihara commented 9 years ago

@ewencp Added label in operations.rst

nehanarkhede commented 9 years ago

@Ishiihara The Monitoring section is empty. Are there useful counters we can add in there or should we just delete that section?

nehanarkhede commented 9 years ago

@Ishiihara The docs are in much better shape. Left a few more comments. After addressing those, feel free to merge

Ishiihara commented 9 years ago

Merge the doc to confluent-master for now. Will expand the design section more and add an architecture diagram.