On 22/02/2012 18:41, Costin Manolache wrote: > On Wed, Feb 22, 2012 at 10:27 AM, Konstantin Kolinko <knst.koli...@gmail.com >> wrote: > >> 2012/2/22 Costin Manolache <cos...@gmail.com>: >>> On Wed, Feb 22, 2012 at 8:57 AM, Mark Thomas <ma...@apache.org> wrote: >>> >>>> On 22/02/2012 16:51, Costin Manolache wrote: >>>>> First part submitted. For the second: what is the process for adding a >>>>> dependency ? >>>> >>>> Why do you need the dependency? >>>> >>> >>> Spdy requires header compression - with a pre-defined dictionary ( >>> containing header names and common values ). It also requires 'flush' - a >>> single compression context is used for all headers in the stream. AFAIK >>> this is not possible with the jdk compress library. >>> >>> That's one of the ways spdy gets its speed - it is not optional in >>> chrome/firefox. >>> >> >> 1. AJP/1.3 protocol has dictionary for header names. >> 2. In Deflater class explicit flush operation was added in Java 1.7. >> > > I think requiring Java1.7 is too much. Most prod servers are on 1.6. > > Would making it an 'optional' dependency work ( i.e. skip compiling the > required > classes / use class for name ) ? I can add an option to use 1.7 Deflater if > available.
If we are talking about Tomcat 8 then that is very likely to require Java 1.7 anyway. 1.6 is close to EOL. Are you considering back-porting this to 7.0.x? There is also a workaround already in 7.0.x for flushing compressed streams. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org