Reviewers: [email protected] Description: Currently the container for accelerate use case is hardcoded as "default". This change adds a new "accel" container and points HtmlAccelServlet and AccelUriManager to use this container. This change should go in after the bug for percolating container information throughout has been resolved. Refer to https://issues.apache.org/jira/browse/SHINDIG-1411 for more info on the bug, and here for relevant codereviews: http://codereview.appspot.com/2004042/ http://codereview.appspot.com/1888046/
Please review this at http://codereview.appspot.com/1949048/ Affected files: config/container.js java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/HtmlAccelServlet.java java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/AccelUriManager.java java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultAccelUriManager.java java/gadgets/src/test/java/org/apache/shindig/gadgets/uri/DefaultAccelUriManagerTest.java Thanks Gagan ---------- Forwarded message ---------- From: <[email protected]> Date: Tue, Aug 24, 2010 at 4:53 PM Subject: Re: Moving HtmlAccelServlet from "default" container to "accel" container (issue1949048) To: [email protected], [email protected] Cc: [email protected] LGTM. http://codereview.appspot.com/1949048/
