Code review comment for lp:~pbeaman/akiban-server/fix-direct-rest-bugs

Revision history for this message
Mike McMahon (mmcm) wrote :

Looks okay but doesn't compile.

There is no org.apache.http.HttpStatus. httpcomponents core isn't in the pom. I think HttpServletResponse will give the necessary SC_ constants, if that's all that's needed.

RestServiceScriptsIT.java does not have a proper header.

review: Needs Fixing

« Back to merge proposal