Thank you! Pushed. Frank
> -----Original Message----- > From: Chris Hegarty [mailto:[email protected]] > Sent: Monday, September 17, 2018 6:16 PM > To: Frank Yuan; 'core-libs-dev' > Subject: Re: RFR (12): 8210819: Update the host name in CNameTest.java > > Looks fine Frank. Thanks, > -Chris. > > On 17/09/18 08:12, Frank Yuan wrote: > > Hi > > > > > > > > Would you like to review the following patch for > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8210819 > > > > > > > > --- a/test/jdk/sun/net/InetAddress/nameservice/dns/CNameTest.java > > Wed Sep 12 21:56:59 2018 -0700 > > > > +++ b/test/jdk/sun/net/InetAddress/nameservice/dns/CNameTest.java Mon > > Sep 17 15:00:55 2018 +0800 > > > > @@ -1,5 +1,5 @@ > > > > /* > > > > - * Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved. > > > > + * Copyright (c) 2017, 2018, Oracle and/or its affiliates. All rights > > reserved. > > > > * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. > > > > * > > > > * This code is free software; you can redistribute it and/or modify it > > > > @@ -38,7 +38,7 @@ > > > > * @summary Test DNS provider's handling of CNAME records > > > > */ > > > > public class CNameTest { > > > > - private static final String HOST = "www-proxy.us.oracle.com"; > > > > + private static final String HOST = "www.w3c.org"; > > > > > > > > CNameTest.java is to test DNS provider's handling of CNAME records, so the > > test requires dns has cname record for the hostname. The > > old name is not appropriate, I would replace it with www.w3c.org > > <http://www.w3c.org> . > > > > > > > > Thanks > > > > Frank > >
