You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Edward Sargisson <es...@pobox.com> on 2013/04/26 23:16:06 UTC

Review Request: Doc update for FLUME-2015, FLUME-1992, FLUME-1972 and FLUME-1782.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10804/
-----------------------------------------------------------

Review request for Flume.


Description
-------

Documentation for all currently known issues:
* Description of index name rolling
* required elasticsearch and lucene core jars
* correct heading level.


Diffs
-----

  flume-ng-doc/sphinx/FlumeUserGuide.rst 38f2205 

Diff: https://reviews.apache.org/r/10804/diff/


Testing
-------

mvn site run and resulting FlumeUserGuide.html checked.


Thanks,

Edward Sargisson


Re: Review Request: Doc update for FLUME-2015, FLUME-1992, FLUME-1972 and FLUME-1782.

Posted by Mike Percy <mp...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10804/#review20429
-----------------------------------------------------------

Ship it!


+1

Looks good, thanks for the docs!

- Mike Percy


On April 26, 2013, 9:16 p.m., Edward Sargisson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10804/
> -----------------------------------------------------------
> 
> (Updated April 26, 2013, 9:16 p.m.)
> 
> 
> Review request for Flume.
> 
> 
> Description
> -------
> 
> Documentation for all currently known issues:
> * Description of index name rolling
> * required elasticsearch and lucene core jars
> * correct heading level.
> 
> 
> Diffs
> -----
> 
>   flume-ng-doc/sphinx/FlumeUserGuide.rst 38f2205 
> 
> Diff: https://reviews.apache.org/r/10804/diff/
> 
> 
> Testing
> -------
> 
> mvn site run and resulting FlumeUserGuide.html checked.
> 
> 
> Thanks,
> 
> Edward Sargisson
> 
>