search

Comcast / sirius-reference-app

A reference application for getting started with Sirius
Apache License 2.0
5 stars 10 forks source link

add README for groovy example #13

Closed tbarker9comcast closed 10 years ago

tbarker9comcast commented 10 years ago

-1 DO NOT MERGE!

This is here so @jwakemen can review it

tbarker9comcast commented 10 years ago

This is ready to merge now

jwakemen commented 10 years ago

This file looks good. Should the -DskipTests flag be removed from the maven command for building Sirius itself?

There is some information that might be missing, like what happens when you run more than the number of nodes that are listed in the cluster configuration file. I understand this is a simple application and some of the stuff I have concerns about might be beyond the scope of the reference application's readme file.