-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51045/#review145641
-----------------------------------------------------------



Is there a downside to including this parameter?  The upside is expected 
behavior of jaxb integration with jersey.  To be honest, I'm not sure how a 
bunch of endpoints continue to work with this patch, as I believe several 
debugging endpoints rely on POJO mapping.

- Bill Farner


On Aug. 12, 2016, 10:23 a.m., Zameer Manji wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51045/
> -----------------------------------------------------------
> 
> (Updated Aug. 12, 2016, 10:23 a.m.)
> 
> 
> Review request for Aurora and Joshua Cohen.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> The JSON/POJO mapping parameter is only required for the API servlet. The 
> other servlets do not need this configuration parameter.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java 
> 7bf8185695ab0af799ee297f39bcd79f6546cf6b 
>   src/main/java/org/apache/aurora/scheduler/http/api/ApiModule.java 
> e46820952fb6028911bca924169ceade6a134bfc 
> 
> Diff: https://reviews.apache.org/r/51045/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Zameer Manji
> 
>

Reply via email to