Thanks! Just curious, what tests did you run and under what environment?
On Tue, Mar 12, 2019 at 10:34 AM Chris Hegarty <chris.hega...@oracle.com> wrote: > Arthur, > > On 12 Mar 2019, at 16:59, Arthur Eubanks <aeuba...@google.com> wrote: > > Thanks for creating the umbrella bug. > > Split long line. > Added the HTTPTestServer.java and UrgentDataTest.java changes in Chris's > suggestions. > > > Thanks. > > The other 2 involve URI/URLs, and there are a lot of those changes, will > group them in a later change. > > > That is true, and it is absolutely fine to leave them until a follow on. > > The reason I suggested the change is because almost all of the other > HTTP Client tests run successfully on an IPv6-only environment. The > reason is that the URL-string when converted to a URI has the square > brackets added by URI.create. Anyway, not critical for now. > > PTAL: http://cr.openjdk.java.net/~aeubanks/8220083/webrev.02/ > > > Looks good. > > You are probably all set, but if you need a sponsor just ask. > > -Chris. >
