Changeset: e77341306d8f
Author:dsamersoff
Date: 2012-08-09 17:42 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/e77341306d8f
7183753: [TEST] Some colon in the diff for this test
Summary: Reference output file contains extra colon
Reviewed-by: sspitsyn, mgronlun
! tes
t; http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/e77341306d8f
>>
>> 7183753: [TEST] Some colon in the diff for this test
>> Summary: Reference output file contains extra colon
>> Reviewed-by: sspitsyn, mgronlun
>>
>> ! test/sun/tools/jcmd/help_help.out
>>
>
>
--
Dmitry Samersoff
Java Hotspot development team, SPB04
* There will come soft rains ...
Problem description:
File with reference output for set of jcmd tests contains extra
colon (:)
Fix description:
colon removed so expected output matches the actual one.
Link to jdk8 changeset:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bf85c3ab2637
-Dmitry
--
Dmitry Samersoff
Java Hotspot
Changeset: 2b19b0c77416
Author:dsamersoff
Date: 2013-01-22 02:34 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/2b19b0c77416
8002048: Protocol to discovery of manageable Java processes on a network
Summary: Introduce a protocol to discover manageble Java instances acr
/bugdatabase/view_bug.do?bug_id=8002048
jep:
http://openjdk.java.net/jeps/168
jdk8 chageset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/962d6612cace
Review:
http://mail.openjdk.java.net/pipermail/serviceability-dev/2012-December/007618.html
Reviewed by:
Staffan Larsen
Daniel Fuchs
--
Dmitry
/bugdatabase/view_bug.do?bug_id=8002048
jep:
http://openjdk.java.net/jeps/168
jdk8 chageset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/962d6612cace
Review:
http://mail.openjdk.java.net/pipermail/serviceability-dev/2012-December/007618.html
Reviewed by:
Staffan Larsen
Daniel Fuchs
--
Dmitry
I'm working on it.
> There's also an issue with the copyright formats. Have you logged a bug
> to track that ? Please log one if not.
Yes the bug is filled.
JDK-8007536 Incorrect copyright header in JDP files
-Dmitry
>
> regards,
> Sean.
>
> On 05/02/2013
/view_bug.do?bug_id=8007277
jdk8 chageset:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0e7d5dd84fdf
Review and discussion:
http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-February/008326.html
Reviewed by:
Alan Bateman
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg
Changeset: 7c0d4bfd9d2c
Author:dsamersoff
Date: 2013-02-06 20:09 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/7c0d4bfd9d2c
8007277: JDK-8002048 testcase fails to compile
Summary: sun.* classes is not included to ct.sym file and symbol file have to
be ignored
Review
On 02/05/2013 04:09 PM, Dmitry Samersoff wrote:
>
>> webrev:
>> http://cr.openjdk.java.net/~dsamersoff/8002048.JDP/webrev/jdk7u-dev/
>
> + private static final String JDP_DEFAULT_ADDRESS = "239.255.255.225";
> + private static final int JDP_DEFAULT_
Florian,
Thank you for doing it. I'll check with client team what is better for
them.
-Dmitry
On 2013-02-11 14:28, Florian Weimer wrote:
> On 02/07/2013 12:02 PM, Dmitry Samersoff wrote:
>> Florian,
>>
>>> I can submit a request to IANA if you prefer, but I don&#x
7536
jdk8 chageset:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1df991184045
Reviewed by:
Mikael Vidstedt
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* Give Rabbit time, and he'll always get the answer
Changeset: c9bc97a00f69
Author:dsamersoff
Date: 2013-02-12 14:03 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/c9bc97a00f69
8007536: Incorrect copyright header in JDP files
Summary: Copyright header in JDP files missed the "classpath exception" rule.
Reviewed-by: mik
bug:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8007786
jdk8 chageset:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f7fb173ac833
Reviewed by:
Staffan Larsen
Sergurei Spitsyn
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* Give Rabbit time, and he
Changeset: 4697f4725cee
Author:dsamersoff
Date: 2013-02-12 16:41 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/4697f4725cee
8007786: JDK-8002048 testcase doesn't work on Solaris
Summary: test built in into Solaris shell doesn't have -e operator
Reviewed-by: sla, sspi
s.
>>
>>
>> On 02/12/2013 12:04 PM, dmitry.samers...@oracle.com wrote:
>>> Changeset: f7fb173ac833
>>> Author:dsamersoff
>>> Date: 2013-02-12 16:02 +0400
>>> URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f7fb173ac833
>>>
&
jdk/rev/f7fb173ac833
>>>
>>> 8007786: JDK-8002048 testcase doesn't work on Solaris
>>> Summary: test built in into Solaris shell doesn't have -e operator
>>> Reviewed-by: sla, sspitsyn
>>>
>>> ! test/sun/management/jdp/JdpTest.sh
>>>
>
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* Give Rabbit time, and he'll always get the answer
tcase doesn't work on Solaris
>> Summary: test built in into Solaris shell doesn't have -e operator
>> Reviewed-by: sla, sspitsyn
>>
>> ! test/sun/management/jdp/JdpTest.sh
>>
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* Give Rabbit time, and he'll always get the answer
t/jdp/JdpTest.sh
...
ACTION: compile -- Passed. Compilation successful
...
ACTION: shell -- Passed. Execution successful
REASON: User specified action: run shell JdpTest.sh --jtreg --no-compile
TIME: 18.209 seconds
-Dmitry
.
>
> Mandy
>
> On 2/12/2013 2:18 PM, Dmitry Samersoff wrote:
&
lasses}
> fi
> @@ -319,7 +319,7 @@ rm -f ${_logname}
> rm -f ${_logname}
> rm -f ${_policyname}
>
> -if [ -e ${_testsrc}/policy.tpl ]
> +if [ -f ${_testsrc}/policy.tpl ]
> then
>
> cat ${_testsrc}/policy.tpl | \
>
> -Chris.
>
> On 12/02/2013 22:19,
}
> fi
> @@ -319,7 +319,7 @@ rm -f ${_logname}
> rm -f ${_logname}
> rm -f ${_policyname}
>
> -if [ -e ${_testsrc}/policy.tpl ]
> +if [ -f ${_testsrc}/policy.tpl ]
> then
>
> cat ${_testsrc}/policy.tpl | \
>
> -Chris.
>
> On 12/02/2013 22:19, Dmitry
jdk8 chageset:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8181be9a3538
Reviewed by:
Chris Hegarty
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* Give Rabbit time, and he'll always get the answer
Changeset: 064aa2ac5ddf
Author:dsamersoff
Date: 2013-02-14 14:03 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/064aa2ac5ddf
8008095: TEST_BUG: JDK-8002048 one more testcase failure on Solaris
Summary: fixed couple of more Solaris shell incompatibilities
Reviewed-by:
Florian,
Thank you very much for doing it.
I'll file the CR and change default numbers.
-Dmitry
On 2013-03-13 23:17, Florian Weimer wrote:
> On 02/11/2013 01:04 PM, Dmitry Samersoff wrote:
>> Thank you for doing it. I'll check with client team what is better for
>>
vase/downloads/index.html[1]
> [2] http://openjdk.java.net/projects/jdk7u/releases/7u14.html[2]
> [3] http://www.oracle.com/technetwork/topics/security/alerts-086861.html[3]
> [4]
> http://mail.openjdk.java.net/pipermail/jdk7u-dev/2013-February/005725.html[4]
>
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* Give Rabbit time, and he'll always get the answer
://hg.openjdk.java.net/jdk8/tl/jdk/rev/7d7bfce34a79
Reviewed-by: dholmes, jbachorik, hirt
Contributed-by: fwei...@redhat.com
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.
Changeset: df02fb90c36d
Author:dsamersoff
Date: 2013-05-29 12:04 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/df02fb90c36d
8014420: Default JDP address does not match the one assigned by IANA
Summary: JDP protocol defaults changed to IANA assigned values
Reviewed-by
entry contains non-ASCII characters
code produces invalid packet.
webrev:
http://cr.openjdk.java.net/~dsamersoff/8015604/webrev.02/
bug:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8015604
jdk8 chageset:
N/A
Reviewed-by: David Holmes, Staffan Larsen, Peter Allwin
-Dmitry
--
D
jdk8 and 7u-dev at the same time, so no
extra records necessary.
-Dmitry
>
> Approved.
>
> regards,
> Sean.
>
> On 04/06/2013 10:55, Dmitry Samersoff wrote:
>> Requesting approval to push the following change to jdk7u40:
>>
>> To meet 7u40 schedule, I&#x
Changeset: 21bf0e775ea4
Author:dsamersoff
Date: 2013-06-05 17:19 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/21bf0e775ea4
8015604: JDP packets containing ideographic characters are broken
Summary: Code uses string length rather than byte array length and non ascii
compatibility.
*webrev*
http://cr.openjdk.java.net/~dsamersoff/8011038/webrev.01/
*jdk8 changeset*
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fce446b29577
*testing*
JCK, no separate regression tests required.
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I
Changeset: 91cdaff2fed3
Author:dsamersoff
Date: 2013-08-06 17:37 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/91cdaff2fed3
8011038: sourceObj validation during desereliazation of RelationNotification
should be relaxed
Summary: sourceObj could be set to null by setS
Hi Everyone,
Please approve:
JDK-8009213 sun/management/jdp/JdpTest.sh fails with exit code 1
webrev:
http://cr.openjdk.java.net/~dsamersoff/JDK-8009213/webrev.01/
-Dmitry
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but
Sean,
Yes.
http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-September/011642.html
-Dmitry
On 2013-10-03 17:09, Seán Coffey wrote:
> Is there a code review link for this backport Dmitry ?
>
> regards,
> Sean.
>
> On 03/10/2013 13:57, Dmitry Samersoff wro
Changeset: 85a94425d233
Author:dsamersoff
Date: 2013-10-03 23:24 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/85a94425d233
8009213: sun/management/jdp/JdpTest.sh fails with exit code 1
Summary: There's no guarantee that the java process has executed far enough to
b
eks of bake time in
> 8 before pushing it to jdk7u-dev? I realize it is tempting to get it
> pushed now while the patch is fresh (and maybe that is okay), it's just
> that all changes in this area have had a history of change behavior in
> subtle ways and these take time to surface.
splay hunan readable progress instead of many dots in jhat
>>>
>> There wasn't any patch attached to your mail but if you are proposing a
>> patch to the jhat tool then the serviceability-dev is the mailing list to
>> use. See also the "How to contribute" page
Hi Everyone,
Please review the fix.
The fix remove a colon (:) from file with reference output.
http://cr.openjdk.java.net/~dsamersoff/JDK-8019800/webrev.01/
-Dmitry
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won&#
Please approve.
On 2014-02-14 22:21, Dmitry Samersoff wrote:
> Hi Everyone,
>
> Please review the fix.
>
> The fix remove a colon (:) from file with reference output.
>
> http://cr.openjdk.java.net/~dsamersoff/JDK-8019800/webrev.01/
>
> -Dmitry
>
--
estcase is listed in ProblemList.txt and hence doesn't run
> through JPRT.
>
> Are you going to remove it from ProblemList if it's now working ? (i.e.
> all JPRT runs pass)
>
> regards,
> Sean.
>
> On 18/02/2014 07:54, Dmitry Samersoff wrote:
>> Pl
Sean,
OK. Thank you for doing it.
-Dmitry.
On 2014-02-18 13:47, Seán Coffey wrote:
> Dmitry,
>
> I've seen this type of testcase fail alot. I'm going to pull down your
> patch and run a JPRT job on it. Will get back to you.
>
> regards,
> Sean.
>
> On 18/
Changeset: 827bc20e8f61
Author:dsamersoff
Date: 2014-02-21 19:26 +0400
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/827bc20e8f61
8019800: Test sun/tools/jcmd/jcmd-f.sh failing after JDK-8017011
Summary: removed extra colon
Reviewed-by: sla, coffeys
! test/ProblemList.txt
command line.
JDK8 changeset:
http://hg.openjdk.java.net/hsx/hsx23/hotspot/rev/3322dd7f0a4d
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1e737abbff6f
List of reviewers:
Mandy Chung
Karen Kinnear
--
Dmitry Samersoff
Java Hotspot development team, SPB04
* There will come soft rains ...
Assert wrongly triggered by invoke dynamic code
http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/ff29ce866f23
Reviewers:
Dan Daugherty
Paul Hohensee
--
Dmitry Samersoff
Java Hotspot development team, SPB04
* There will come soft rains ...
44 matches
Mail list logo
