Mike Percy has posted comments on this change.

Change subject: Add 0.7.1 docs
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/2515/1/releases/0.7.1/index.md
File releases/0.7.1/index.md:

Line 35: * [Kudu 0.7.1 source 
tarball](http://www.apache.org/dyn/closer.lua/incubator/kudu/0.7.1) 
[MD5](http://www.apache.org/dist/incubator/kudu/0.7.1/apache-kudu-incubating-0.7.1.tar.gz.md5)
 
[Signature](http://www.apache.org/dist/incubator/kudu/0.7.1/apache-kudu-incubating-0.7.1.tar.gz.asc)
Nit: Can we use HTTPS, at least for the sigs / checksums? Also, you are missing 
the SHA checksum. Also, I think it would be easier to read if it looked like 
"Kudu 0.7.1 source tarball (MD5, SHA1, Signature)" ... right now there are no 
parentheses.


http://gerrit.cloudera.org:8080/#/c/2515/1/releases/index.md
File releases/index.md:

Line 15:  
nit: trailing single space


Line 19: * [Kudu 0.7.0 (beta)](0.7.0/) was released on Feb 26, 2016. It is the 
latest version of Kudu.  
Not the latest version of Kudu


-- 
To view, visit http://gerrit.cloudera.org:8080/2515
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id120a723e45863da8eb24219a01f5d9bb7fef24c
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-HasComments: Yes

Reply via email to