Committed as r966049. Feel free to close out this review.

On 2010/07/20 22:42:26, johnfargo wrote:
http://codereview.appspot.com/1696052/diff/6001/7001
File

java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandler.java
(right):

http://codereview.appspot.com/1696052/diff/6001/7001#newcode123

java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandler.java:123:
CompletionService<R> completionService = new
ExecutorCompletionService<R>(executor);
why not just init this in GadgetHandler at the top level?



http://codereview.appspot.com/1696052/show

Reply via email to