[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-net-dev
Subject: =?UTF-8?B?UkZSIChYUykgODIzNzgwNjogQ29udmVydCBUSU1FT1VUIG9mIHN1bi5uZXQuZG5zLlJlc29s?= =?UTF-8?B?dmVyQ
From: "Denghui Dong" <denghui.ddh () alibaba-inc ! com>
Date: 2020-01-24 8:36:03
Message-ID: b69aeb95-cb09-486e-9e9a-2503d4eb8c3a.denghui.ddh () alibaba-inc ! com
[Download RAW message or body]
[Attachment #2 (text/plain)]
Hi team,
Could I have a review of a small change that converts TIMEOUT of \
ResolverConfigurationImpl into property. Summary:
The value of TIMEOUT of ResolverConfigurationImpl was hardcoded, it's useful to use a \
system property to specify the value of it.
Bug: https://bugs.openjdk.java.net/browse/JDK-8237806
CSR: https://bugs.openjdk.java.net/browse/JDK-8237815
Webrev: http://cr.openjdk.java.net/~ddong/8237806/webrev.00/
Thanks
Denghui Dong
[Attachment #3 (text/html)]
<div class="__aliyun_email_body_block"><div style="clear:both;"><div \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><span \
class=" __aliyun_node_has_color" \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;font-family:Microsoft \
Yahei;font-size:14.0px;color:#000000;">Hi team,</span></div><div \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><span \
class=" __aliyun_node_has_color" \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;font-family:Microsoft \
Yahei;font-size:14.0px;color:#000000;"><span class=" __aliyun_node_has_color" \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline;">Could I have a review of a \
small change that converts TIMEOUT of ResolverConfigurationImpl into \
property.</span></span></div><div \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;">Summary:</div><div \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;">The \
value of TIMEOUT of <span class=" __aliyun_node_has_color \
__aliyun_node_has_bgcolor" \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;background-color:#ffffff;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline;">ResolverConfigurationImpl \
was hardcoded, it's useful to use a system property to specify</span></div><div \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><span \
class=" __aliyun_node_has_color __aliyun_node_has_bgcolor" \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;background-color:#ffffff;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline;">the \
value of it.</span></div><div \
style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family:Tah \
oma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-varian \
t-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-transform:none;wi \
dows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><br \
></div><div style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;fo \
> nt-family:Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:nor \
> mal;font-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text- \
> transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><br \
> ></div><div style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000; \
> > font-family:Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures \
> > :normal;font-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px \
> > ;text-transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;">Bug: <a \
> > href="https://bugs.openjdk.java.net/browse/JDK-8237806" \
> > target="_blank">https://bugs.openjdk.java.net/browse/JDK-8237806</a></div></div><div \
> > style="clear:both;"><div \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;fon \
> > t-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-trans \
> > form:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><a \
> > href="https://bugs.openjdk.java.net/browse/JDK-8237806" \
> > target="_blank"></a></div><div \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;fon \
> > t-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-trans \
> > form:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><div \
> > class=" __aliyun_node_has_color" \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;font-style:normal;font-variant-ligat \
> > ures:normal;font-variant-caps:normal;font-weight:400;text-align:start;text-indent: \
> > .0px;text-transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;">CSR: <a \
> > href="https://bugs.openjdk.java.net/browse/JDK-8237815">https://bugs.openjdk.java.net/browse/JDK-8237815</a></div></div></div><div \
> > style="clear:both;"><div \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;fon \
> > t-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-trans \
> > form:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><div \
> > class=" __aliyun_node_has_color" \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;font-style:normal;font-variant-ligat \
> > ures:normal;font-variant-caps:normal;font-weight:400;text-align:start;text-indent: \
> > .0px;text-transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;">Webrev: <a \
> > href="http://cr.openjdk.java.net/~ddong/8237806/webrev.00/" \
> > target="_blank">http://cr.openjdk.java.net/~ddong/8237806/webrev.00/</a></div></div></div><div \
> > style="clear:both;"><div \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;fon \
> > t-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-trans \
> > form:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><div \
> > class=" __aliyun_node_has_color" \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;font-style:normal;font-variant-ligat \
> > ures:normal;font-variant-caps:normal;font-weight:400;text-align:start;text-indent: \
> > .0px;text-transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><a \
> > href="http://cr.openjdk.java.net/~ddong/8237806/webrev.00/" \
> > target="_blank"></a></div></div><div \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;fon \
> > t-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text-trans \
> > form:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><div \
> > class=" __aliyun_node_has_color" \
> > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-family \
> > :Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;font-style:normal;font-variant-ligat \
> > ures:normal;font-variant-caps:normal;font-weight:400;text-align:start;text-indent: \
> > .0px;text-transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><br \
> > ></div></div><div \
> > > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-fami \
> > > ly:Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal \
> > > ;font-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text \
> > > -transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><span \
> > > class=" __aliyun_node_has_color" \
> > > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;font-family:Microsoft \
> > > Yahei;font-size:14.0px;color:#000000;">Thanks</span></div><div \
> > > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;color:#000000;font-fami \
> > > ly:Tahoma,Arial;font-size:14.0px;font-style:normal;font-variant-ligatures:normal \
> > > ;font-variant-caps:normal;font-weight:400;text-align:start;text-indent:.0px;text \
> > > -transform:none;widows:2;text-decoration-style:initial;text-decoration-color:initial;clear:both;"><span \
> > > class=" __aliyun_node_has_color" \
> > > style="margin:.0px;padding:.0px;border:.0px;outline:.0px;font-family:Microsoft \
> > > Yahei;font-size:14.0px;color:#000000;">Denghui Dong</span></div></div><div \
> > > style="clear:both;"><span style="font-family:Microsoft \
> > > Yahei;font-size:14.0px;color:#000000;"><br ></span></div></div>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic