You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@fluo.apache.org by GitBox <gi...@apache.org> on 2018/08/28 21:56:59 UTC

[GitHub] ctubbsii commented on a change in pull request #149: Fluo recipes updates

ctubbsii commented on a change in pull request #149: Fluo recipes updates
URL: https://github.com/apache/fluo-recipes/pull/149#discussion_r213485569
 
 

 ##########
 File path: modules/test/src/main/java/org/apache/fluo/recipes/test/AccumuloExportITBase.java
 ##########
 @@ -39,7 +39,8 @@
 /**
  * This class is intended to be extended by classes testing exporting from Fluo to Accumulo. Using
  * MiniFluo by itself is easy. However, using MiniAccumulo and MiniFluo together involves writing a
- * lot of boiler plate code. Thats why this class exists, its a place to put that boiler plate code.
+ * lot of boiler plate code. That's why this class exists, its a place to put that boiler plate
 
 Review comment:
   boilerplate is one word. There's an opportunity to clean that up here also, which would probably keep "code" on the same line.
   
   Also, "exists, its a place" should be replaced with "exists: it's a place"

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services