giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alessandro Presta" <alessan...@fb.com>
Subject Re: Review Request: Improve website for upcoming release
Date Tue, 09 Apr 2013 17:39:06 GMT

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


Looks much nicer!
Some random comments:
- I would try to reduce the amount of text on the landing page as much as possible: a few
key sentences on the project (reference BSP, Pregel, Hadoop), how to build, and maybe a code
snippet to show the vertex API. Everything else could probably be moved down or to other pages.
- The logo wastes quite a lot of space the way it's positioned (look at all the white space
to its right). It would be better if it was side by side with the title or other text.
- There are some links to content that's probably outdated (e.g, the shortest paths example,
the quick start guide). The menus could probably be better organized: we could separate user-oriented
info (summary, license, mailing lists etc) from developer info (how to build the site, continuous
integration, etc). The Modules section seems useless right now. License should probably link
to the specific version. Do we even use all those Reports?
- The PageRank example currently has no code snippets. Also, in my opinion it's sufficient
to call it PageRank (the Hello sounds a bit weird to me...).

- Alessandro Presta


On April 9, 2013, 7:28 a.m., Avery Ching wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10365/
> -----------------------------------------------------------
> 
> (Updated April 9, 2013, 7:28 a.m.)
> 
> 
> Review request for giraph.
> 
> 
> Description
> -------
> 
> The website was in a horrible state (didn't even build).  I fixed this and ported it
to a modern skin (fluido).
> 
> * Google +1 / Facebook like support
> * Google site search support
> * Modified ConfigurationUtils.java to support a non-munged profile
> * Updated incubator references
> * Added hudson location
> * Site now is a multi-module site instead of giraph-core only
> * Changed site layout to be more modern using the fluido skin
> * Added instructions on building the site
> * Added initial Hello, Page Rank example 
> * Here's what the site looks like now http://apachegiraph.zapto.org/~aching/staging/
> 
> 
> This addresses bug GIRAPH-612.
>     https://issues.apache.org/jira/browse/GIRAPH-612
> 
> 
> Diffs
> -----
> 
>   giraph-accumulo/pom.xml fd1ee2beb170b8c8cc141c4c9f6821f78b2476c6 
>   giraph-core/pom.xml 2f473ed764f94e61aa3bac50287424e64a13750d 
>   giraph-core/src/main/java/org/apache/giraph/utils/ConfigurationUtils.java 9ebe69301e2c42eb4d96c9916d4acb72ae795fa2

>   giraph-core/src/site/resources/css/site.css bd3e914cb7e3a13ee85e821fb2d73650077c4a03

>   giraph-core/src/site/resources/images/ApacheGiraph.svg 8c339c8fea13be1ed5d6280c61fe9271e636917f

>   giraph-core/src/site/site.xml 24b7ab25ed2103c6c96e68d7e73d64977a2851cb 
>   giraph-core/src/site/xdoc/index.xml 2d496a229ae0db933c0a94047ec0c4dddfebabe4 
>   giraph-examples/pom.xml 21e8ccfc20042b14fcd2059272522001d566e716 
>   giraph-hbase/pom.xml 41b27e82aac24ded3ccb06eba527f5f8cf70bceb 
>   giraph-hcatalog/pom.xml ee5710a67fcda5fdc5a77e9cdf4a2fc9e9722c88 
>   giraph-hive/pom.xml 2f80165ff6ecd4f255f8c6e9cd3acdf07c872817 
>   pom.xml 71467cdc1ef83ad835ab7522e6112e8c1389a772 
>   src/site/resources/css/site.css PRE-CREATION 
>   src/site/resources/images/ApacheGiraph.svg PRE-CREATION 
>   src/site/site.xml PRE-CREATION 
>   src/site/xdoc/build_site.xml PRE-CREATION 
>   src/site/xdoc/hello_pagerank.xml PRE-CREATION 
>   src/site/xdoc/index.xml PRE-CREATION 
>   src/site/xdoc/javadoc_modules.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10365/diff/
> 
> 
> Testing
> -------
> 
> Built the site.
> 
> 
> Thanks,
> 
> Avery Ching
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message