You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@harmony.apache.org by Alexei Fedotov <al...@gmail.com> on 2007/10/01 14:29:35 UTC

[bti][p-unit] finally, it is ready

Hello,

The patch for the stress test suite which make use of p-unit
performace infrastructure for comparative stress load analysis is
ready  [1]. I want to ask a commuity when it might be committed.

Thanks, Alexei

[1] http://issues.apache.org/jira/browse/HARMONY-4114

P.S. Special thanks to Andrew Zhang for his support.

On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> Alexey,
> Thanks for reference!
[...]
-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Fedotov <al...@gmail.com>.
Hello Stepan,
I'm gratefull for your interest to this patch.

>  why did you remove logging for "date-of-creation" and
> "authors" field?

I believe these fields are now excessive. A date of creation may be
got from svn as well as an author. I was not a supporter or author
removal idea because svn lists only committers, but this was accepted
anyway, and now we should to follow.

I agree that obsolete warnings may be removed by decreasing logging
level, but this is not a correct way to follow to my personal point of
view.

> Is is necessary for p-unit integration? I guess no -
> you can change logging level (or fix tests description files) to avoid
> unwanted logging.

And you are correct. It is not necessary for p-unit integration. The
goal is not p-unit integration at all. The goal is adding new tests
and integrating them seamlessly, and absence of obsolete warnings is
important for seamless integration.

Thanks!

On 10/29/07, Stepan Mishura <st...@gmail.com> wrote:
> On 10/26/07, Alexei Fedotov <al...@gmail.com> wrote:
> > Stepan, sure. Thank you for your interest to this patch.
> >
> > For reviewer convinience the harness patch is split into two:
> > Harmony-style formatting changes were separated from removing two
> > warning outputs and authorship [1]. While I respect a contribution of
> > Andrey Tyuryushkin, I have learned the the author removal trick from
> > Sveta.
> >
>
> Alexei,
>
> Thanks for providing new patch that free from formatting changes -
> that made the review easier. Now I have a question to your updates to
> the harness: why did you remove logging for "date-of-creation" and
> "authors" field? Is is necessary for p-unit integration? I guess no -
> you can change logging level (or fix tests description files) to avoid
> unwanted logging.
>
> Thanks,
> Stepan.
>
> > Thanks.
> >
> > [1] http://issues.apache.org/jira/secure/attachment/12368462/harness_review.patch
>
> <SNIP>
>


-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Stepan Mishura <st...@gmail.com>.
On 10/26/07, Alexei Fedotov <al...@gmail.com> wrote:
> Stepan, sure. Thank you for your interest to this patch.
>
> For reviewer convinience the harness patch is split into two:
> Harmony-style formatting changes were separated from removing two
> warning outputs and authorship [1]. While I respect a contribution of
> Andrey Tyuryushkin, I have learned the the author removal trick from
> Sveta.
>

Alexei,

Thanks for providing new patch that free from formatting changes -
that made the review easier. Now I have a question to your updates to
the harness: why did you remove logging for "date-of-creation" and
"authors" field? Is is necessary for p-unit integration? I guess no -
you can change logging level (or fix tests description files) to avoid
unwanted logging.

Thanks,
Stepan.

> Thanks.
>
> [1] http://issues.apache.org/jira/secure/attachment/12368462/harness_review.patch

<SNIP>

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Fedotov <al...@gmail.com>.
Stepan, sure. Thank you for your interest to this patch.

For reviewer convinience the harness patch is split into two:
Harmony-style formatting changes were separated from removing two
warning outputs and authorship [1]. While I respect a contribution of
Andrey Tyuryushkin, I have learned the the author removal trick from
Sveta.

Thanks.

[1] http://issues.apache.org/jira/secure/attachment/12368462/harness_review.patch

On 10/26/07, Stepan Mishura <st...@gmail.com> wrote:
> On 10/25/07, Andrew Zhang <zh...@gmail.com> wrote:
> > On 10/23/07, Alexei Fedotov <al...@gmail.com> wrote:
> > >
> > > Hello Stepan,
> > >
> > > As per our dicsussion I've created a JIRA issue with a distinctive
> > > name [1]. Could one please close HARMONY-4114 [2]?
> >
>
> Hi, Andrew,
>
> I have a look into HARMONY-4992 - looking into diffs it is hard for me
> to get an idea why the harness have to be patched to integrate p-unit.
>
> Alexei, could you comment your update?
>
> Thanks,
> Stepan.
>


-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Stepan Mishura <st...@gmail.com>.
On 10/27/07, Andrew Zhang <zh...@gmail.com> wrote:
> On 10/26/07, Stepan Mishura <st...@gmail.com> wrote:
> >
> > On 10/26/07, Stepan Mishura <st...@gmail.com> wrote:
> > > On 10/25/07, Andrew Zhang <zh...@gmail.com> wrote:
> > > > On 10/23/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > >
> > > > > Hello Stepan,
> > > > >
> > > > > As per our dicsussion I've created a JIRA issue with a distinctive
> > > > > name [1]. Could one please close HARMONY-4114 [2]?
> > > >
> > >
> > > Hi, Andrew,
> > >
> > > I have a look into HARMONY-4992 - looking into diffs it is hard for me
> > > to get an idea why the harness have to be patched to integrate p-unit.
> > >
> >
> > Andrew,
> >
> > Could you take care of HARMONY-4992? I'm quite busy with BTI related
> > tasks and I assume that it is easier for you to understand 1Mb patch
> > for the stress suite which make use of p-unit.
>
>
> Hi Stepan,
>
> Sure, I'll take Harmony-4992.  Thanks!
>

Hi Andrew,

I've tried to setup/run stress suite from scratch but it failed
because of incompatible changes in PUnit framework. I found that
stress suite works OK with r278 but fails to compile with the last
version from PUnit SVN with the log below. So I implemented workaround
for the issue [1] and I'd like to ask you to think about creating a
bundle for using it by stress suite or updating stress suite to
incorporate latest changes in PUnit. Is this possible?

[1] http://svn.apache.org/viewvc?view=rev&revision=655770

build.java:
    [javac] Compiling 785 source files to
C:\cc\i1.6\cc-jre\build\classes\stress\classes
    [javac] C:\cc\i1.6\cc-jre\build\checkouts\stress\qa\src\test\stress\org\apache\harmony\test\share\stress\AbstractTest.java:93:
cannot find symbol
    [javac] symbol  : method releaseWilderness()
    [javac] location: class org.punit.util.MemoryUtil
    [javac]             MemoryUtil.releaseWilderness();
    [javac]                       ^
    [javac] C:\cc\i1.6\cc-jre\build\checkouts\stress\qa\src\test\stress\org\apache\harmony\test\share\stress\AbstractTest.java:95:
cannot find symbol
    [javac] symbol  : method allocateWilderness()
    [javac] location: class org.punit.util.MemoryUtil
    [javac]             MemoryUtil.allocateWilderness();
    [javac]                       ^
    [javac] C:\cc\i1.6\cc-jre\build\checkouts\stress\qa\src\test\stress\org\apache\harmony\test\share\stress\AbstractTest.java:100:
cannot find symbol
    [javac] symbol  : method releaseWilderness()
    [javac] location: class org.punit.util.MemoryUtil
    [javac]         MemoryUtil.releaseWilderness();
    [javac]                   ^
    [javac] C:\cc\i1.6\cc-jre\build\checkouts\stress\qa\src\test\stress\org\apache\harmony\test\share\stress\AbstractTest.java:103:
cannot find symbol
    [javac] symbol  : method allocateWilderness()
    [javac] location: class org.punit.util.MemoryUtil
    [javac]         MemoryUtil.allocateWilderness();
    [javac]                   ^
    [javac] Note: Some input files use or override a deprecated API.
    [javac] Note: Recompile with -Xlint:deprecation for details.
    [javac] Note: Some input files use unchecked or unsafe operations.
    [javac] Note: Recompile with -Xlint:unchecked for details.
    [javac] 4 errors

BUILD FAILED

Thanks,
Stepan.

>
> If have free time I'll
> > take the patch for the harness for testing and if there is any issue
> > with it I'll let you know.
> >
> > Thanks,
> > Stepan.
> >
> > > Alexei, could you comment your update?
> > >
> > > Thanks,
> > > Stepan.
> >
>
>
>
> --
> Best regards,
> Andrew Zhang
>
> http://zhanghuangzhu.blogspot.com/
>

Re: [bti][p-unit] finally, it is ready

Posted by Andrew Zhang <zh...@gmail.com>.
On 10/26/07, Stepan Mishura <st...@gmail.com> wrote:
>
> On 10/26/07, Stepan Mishura <st...@gmail.com> wrote:
> > On 10/25/07, Andrew Zhang <zh...@gmail.com> wrote:
> > > On 10/23/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > >
> > > > Hello Stepan,
> > > >
> > > > As per our dicsussion I've created a JIRA issue with a distinctive
> > > > name [1]. Could one please close HARMONY-4114 [2]?
> > >
> >
> > Hi, Andrew,
> >
> > I have a look into HARMONY-4992 - looking into diffs it is hard for me
> > to get an idea why the harness have to be patched to integrate p-unit.
> >
>
> Andrew,
>
> Could you take care of HARMONY-4992? I'm quite busy with BTI related
> tasks and I assume that it is easier for you to understand 1Mb patch
> for the stress suite which make use of p-unit.


Hi Stepan,

Sure, I'll take Harmony-4992.  Thanks!


If have free time I'll
> take the patch for the harness for testing and if there is any issue
> with it I'll let you know.
>
> Thanks,
> Stepan.
>
> > Alexei, could you comment your update?
> >
> > Thanks,
> > Stepan.
>



-- 
Best regards,
Andrew Zhang

http://zhanghuangzhu.blogspot.com/

Re: [bti][p-unit] finally, it is ready

Posted by Stepan Mishura <st...@gmail.com>.
On 10/26/07, Stepan Mishura <st...@gmail.com> wrote:
> On 10/25/07, Andrew Zhang <zh...@gmail.com> wrote:
> > On 10/23/07, Alexei Fedotov <al...@gmail.com> wrote:
> > >
> > > Hello Stepan,
> > >
> > > As per our dicsussion I've created a JIRA issue with a distinctive
> > > name [1]. Could one please close HARMONY-4114 [2]?
> >
>
> Hi, Andrew,
>
> I have a look into HARMONY-4992 - looking into diffs it is hard for me
> to get an idea why the harness have to be patched to integrate p-unit.
>

Andrew,

Could you take care of HARMONY-4992? I'm quite busy with BTI related
tasks and I assume that it is easier for you to understand 1Mb patch
for the stress suite which make use of p-unit. If have free time I'll
take the patch for the harness for testing and if there is any issue
with it I'll let you know.

Thanks,
Stepan.

> Alexei, could you comment your update?
>
> Thanks,
> Stepan.

Re: [bti][p-unit] finally, it is ready

Posted by Stepan Mishura <st...@gmail.com>.
On 10/25/07, Andrew Zhang <zh...@gmail.com> wrote:
> On 10/23/07, Alexei Fedotov <al...@gmail.com> wrote:
> >
> > Hello Stepan,
> >
> > As per our dicsussion I've created a JIRA issue with a distinctive
> > name [1]. Could one please close HARMONY-4114 [2]?
>

Hi, Andrew,

I have a look into HARMONY-4992 - looking into diffs it is hard for me
to get an idea why the harness have to be patched to integrate p-unit.

Alexei, could you comment your update?

Thanks,
Stepan.

Re: [bti][p-unit] finally, it is ready

Posted by Andrew Zhang <zh...@gmail.com>.
On 10/23/07, Alexei Fedotov <al...@gmail.com> wrote:
>
> Hello Stepan,
>
> As per our dicsussion I've created a JIRA issue with a distinctive
> name [1]. Could one please close HARMONY-4114 [2]?


Hi Stepan,

Are you still looking at this issue? How do you think about the new patch?
Thanks!

Thanks.
>
> [1] http://issues.apache.org/jira/browse/HARMONY-4992
> [2] http://issues.apache.org/jira/browse/HARMONY-4114
>
>
> On 10/16/07, Stepan Mishura <st...@gmail.com> wrote:
> > On 10/15/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > Stepan,
> > > Thank you for your attention to this issue.
> > >
> > > Let me define a term we are using first. "VM specific behavior"
> > > actually is an abstract behavior which is VM specific, and not a
> > > behavior, which is specific for DRLVM. To make tests flexible for any
> > > behavior, I've made them looking like performance tests. For reporting
> > > performance-like metrics p-unit was a viable alternative, see the wiki
> > > page [1] for comparison. So p-unit integration is not something
> > > separate from the stress test suite: it suites actual needs.
> > >
> >
> > Alexei,
> >
> > I'd like to have clear distinction between HARMONY-4114, (according to
> > the its description - "VM intermittently throws *UNEXPECTED*
> > OutOfMemoryError on the test which defines heap fragmentation") and
> > p-unit integration.
> >
> > IMHO we shouldn't mix patches for different tasks/issues in one JIRA.
> >
> > > From technical point of view the patch may be split into several
> > > patches. There is even more interesting alternative to get p-unit as
> > > an external dependency for our repository.
> > >
> >
> > Yes, please separate them if that possible - it will make easier
> > understanding changes. Also please provide some relevant comments for
> > updates. Otherwise, it is hard to understand from comments below that
> > the patch integrates p-unit:
> >
> > "28/Sep/07 11:08 AM:
> > should be applied:
> > 5. harness_test_handler.patch (30 kb)
> > 7. stress.patch (1.12 Mb)
> >
> > should be executed:
> > 1. add_remove.sh (8 kb) "
> >
> > Thanks,
> > Stepan.
> >
> > > Thanks.
> > >
> > > [1] http://wiki.apache.org/harmony/Harmony_Test_Harness_Comparison
> > >
> > > On 10/15/07, Stepan Mishura <st...@gmail.com> wrote:
> > > > Alexei,
> > > >
> > > > Why you attached patches for integrating p-unit to HARMONY-4114 that
> > > > primary aimed to fix specific VM behaviour? IMHO p-unit integration
> a
> > > > different story and needs a separate JIRA to keep things clear.
> Agree?
> > > >
> > > > Thanks,
> > > > Stepan.
> > > >
> > <SNIP>
> >
>
>
> --
> With best regards,
> Alexei,
> ESSD, Intel
>



-- 
Best regards,
Andrew Zhang

http://zhanghuangzhu.blogspot.com/

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Fedotov <al...@gmail.com>.
Hello Stepan,

As per our dicsussion I've created a JIRA issue with a distinctive
name [1]. Could one please close HARMONY-4114 [2]?

Thanks.

[1] http://issues.apache.org/jira/browse/HARMONY-4992
[2] http://issues.apache.org/jira/browse/HARMONY-4114


On 10/16/07, Stepan Mishura <st...@gmail.com> wrote:
> On 10/15/07, Alexei Fedotov <al...@gmail.com> wrote:
> > Stepan,
> > Thank you for your attention to this issue.
> >
> > Let me define a term we are using first. "VM specific behavior"
> > actually is an abstract behavior which is VM specific, and not a
> > behavior, which is specific for DRLVM. To make tests flexible for any
> > behavior, I've made them looking like performance tests. For reporting
> > performance-like metrics p-unit was a viable alternative, see the wiki
> > page [1] for comparison. So p-unit integration is not something
> > separate from the stress test suite: it suites actual needs.
> >
>
> Alexei,
>
> I'd like to have clear distinction between HARMONY-4114, (according to
> the its description - "VM intermittently throws *UNEXPECTED*
> OutOfMemoryError on the test which defines heap fragmentation") and
> p-unit integration.
>
> IMHO we shouldn't mix patches for different tasks/issues in one JIRA.
>
> > From technical point of view the patch may be split into several
> > patches. There is even more interesting alternative to get p-unit as
> > an external dependency for our repository.
> >
>
> Yes, please separate them if that possible - it will make easier
> understanding changes. Also please provide some relevant comments for
> updates. Otherwise, it is hard to understand from comments below that
> the patch integrates p-unit:
>
> "28/Sep/07 11:08 AM:
> should be applied:
> 5. harness_test_handler.patch (30 kb)
> 7. stress.patch (1.12 Mb)
>
> should be executed:
> 1. add_remove.sh (8 kb) "
>
> Thanks,
> Stepan.
>
> > Thanks.
> >
> > [1] http://wiki.apache.org/harmony/Harmony_Test_Harness_Comparison
> >
> > On 10/15/07, Stepan Mishura <st...@gmail.com> wrote:
> > > Alexei,
> > >
> > > Why you attached patches for integrating p-unit to HARMONY-4114 that
> > > primary aimed to fix specific VM behaviour? IMHO p-unit integration a
> > > different story and needs a separate JIRA to keep things clear. Agree?
> > >
> > > Thanks,
> > > Stepan.
> > >
> <SNIP>
>


-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Stepan Mishura <st...@gmail.com>.
On 10/15/07, Alexei Fedotov <al...@gmail.com> wrote:
> Stepan,
> Thank you for your attention to this issue.
>
> Let me define a term we are using first. "VM specific behavior"
> actually is an abstract behavior which is VM specific, and not a
> behavior, which is specific for DRLVM. To make tests flexible for any
> behavior, I've made them looking like performance tests. For reporting
> performance-like metrics p-unit was a viable alternative, see the wiki
> page [1] for comparison. So p-unit integration is not something
> separate from the stress test suite: it suites actual needs.
>

Alexei,

I'd like to have clear distinction between HARMONY-4114, (according to
the its description - "VM intermittently throws *UNEXPECTED*
OutOfMemoryError on the test which defines heap fragmentation") and
p-unit integration.

IMHO we shouldn't mix patches for different tasks/issues in one JIRA.

> From technical point of view the patch may be split into several
> patches. There is even more interesting alternative to get p-unit as
> an external dependency for our repository.
>

Yes, please separate them if that possible - it will make easier
understanding changes. Also please provide some relevant comments for
updates. Otherwise, it is hard to understand from comments below that
the patch integrates p-unit:

"28/Sep/07 11:08 AM:
should be applied:
5. harness_test_handler.patch (30 kb)
7. stress.patch (1.12 Mb)

should be executed:
1. add_remove.sh (8 kb) "

Thanks,
Stepan.

> Thanks.
>
> [1] http://wiki.apache.org/harmony/Harmony_Test_Harness_Comparison
>
> On 10/15/07, Stepan Mishura <st...@gmail.com> wrote:
> > Alexei,
> >
> > Why you attached patches for integrating p-unit to HARMONY-4114 that
> > primary aimed to fix specific VM behaviour? IMHO p-unit integration a
> > different story and needs a separate JIRA to keep things clear. Agree?
> >
> > Thanks,
> > Stepan.
> >
<SNIP>

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Fedotov <al...@gmail.com>.
Stepan,
Thank you for your attention to this issue.

Let me define a term we are using first. "VM specific behavior"
actually is an abstract behavior which is VM specific, and not a
behavior, which is specific for DRLVM. To make tests flexible for any
behavior, I've made them looking like performance tests. For reporting
performance-like metrics p-unit was a viable alternative, see the wiki
page [1] for comparison. So p-unit integration is not something
separate from the stress test suite: it suites actual needs.

>From technical point of view the patch may be split into several
patches. There is even more interesting alternative to get p-unit as
an external dependency for our repository.

Thanks.

[1] http://wiki.apache.org/harmony/Harmony_Test_Harness_Comparison

On 10/15/07, Stepan Mishura <st...@gmail.com> wrote:
> Alexei,
>
> Why you attached patches for integrating p-unit to HARMONY-4114 that
> primary aimed to fix specific VM behaviour? IMHO p-unit integration a
> different story and needs a separate JIRA to keep things clear. Agree?
>
> Thanks,
> Stepan.
>
> On 10/9/07, Alexei Fedotov <al...@gmail.com> wrote:
> > Dear committers,
> >
> > Please excuse me for being persistent. I continue updating this thread
> > to keep it at the top of your mailboxes since I want one of you to get
> > a honorable responsibility of supporting a stress test suite.
> >
> > To check changes one my use CruiseControl interface:
> > ant -Dtest.suites="hdk,stress" install setup run
> >
> > With best regards, Alexei
> >
> > On 10/6/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > Hello all,
> > >
> > > I want to thank participants of this discussion for their interest.
> > > Since the trunk is unfrozen I also want to revive a discussion and get
> > > someone interested in committing the changes [1].
> > >
> > > BTW, do you think it might be feasible to include p-unit as an
> > > external repository by modifying svn:externals property at
> > > http://svn.apache.org/repos/asf/harmony/enhanced/buildtest/branches/2.0/tests/tools/
> > > instead of fetching it from my build script?
> > >
> > > Thanks!
> > >
> > > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> > >
> > >
> > > On 10/2/07, Alexei Zakharov <al...@gmail.com> wrote:
> > > > +1
> > > >
> > > > 2007/10/2, Stepan Mishura <st...@gmail.com>:
> > > > > On 10/2/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > > Hello,
> > > > > >
> > > > > > Could anyone be so kind to review my changes to the test harness [1]?
> > > > > > I believe they are pretty harmless.
> > > > > >
> > > > >
> > > > > I think we shouldn't hurry with the update for the harness during code
> > > > > freeze - there are several suites that depend on the harness. IMO
> > > > > should be done after M3 release.
> > > > >
> > > > > Thanks,
> > > > > Stepan.
> > > > >
> > > > > > Thank you in advance!
> > > > > >
> > > > > > [1] http://issues.apache.org/jira/secure/attachment/12366764/harness_test_handler.patch
> > > > > >
> > > > > > On 10/1/07, Andrew Zhang <zh...@gmail.com> wrote:
> > > > > > > On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > > > >
> > > > > > > > Hello,
> > > > > > > >
> > > > > > > > The patch for the stress test suite which make use of p-unit
> > > > > > > > performace infrastructure for comparative stress load analysis is
> > > > > > > > ready  [1]. I want to ask a commuity when it might be committed.
> > > > > > >
> > > > > > >
> > > > > > > Hi Alexei,
> > > > > > >
> > > > > > > Great work!  Super thanks to YOU!
> > > > > > >
> > > > > > > Thanks, Alexei
> > > > > > > >
> > > > > > > > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> > > > > > > >
> > > > > > > > P.S. Special thanks to Andrew Zhang for his support.
> > > > > > > >
> > > > > > > > On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > > > > > Alexey,
> > > > > > > > > Thanks for reference!
> > > > > > > > [...]
> > > > > > > > --
> > > > > > > > With best regards,
> > > > > > > > Alexei,
> > > > > > > > ESSD, Intel
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Best regards,
> > > > > > > Andrew Zhang
> > > > > > >
> > > > > > > http://zhanghuangzhu.blogspot.com/
> > > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > With best regards,
> > > > > > Alexei,
> > > > > > ESSD, Intel
> > > > > >
> > > > >
> > > >
> > > >
> > > > --
> > > > Alexei Zakharov,
> > > > Intel ESSD
> > > >
> > >
> > >
> > > --
> > > With best regards,
> > > Alexei,
> > > ESSD, Intel
> > >
> >
> >
> > --
> > With best regards,
> > Alexei,
> > ESSD, Intel
>


-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Stepan Mishura <st...@gmail.com>.
Alexei,

Why you attached patches for integrating p-unit to HARMONY-4114 that
primary aimed to fix specific VM behaviour? IMHO p-unit integration a
different story and needs a separate JIRA to keep things clear. Agree?

Thanks,
Stepan.

On 10/9/07, Alexei Fedotov <al...@gmail.com> wrote:
> Dear committers,
>
> Please excuse me for being persistent. I continue updating this thread
> to keep it at the top of your mailboxes since I want one of you to get
> a honorable responsibility of supporting a stress test suite.
>
> To check changes one my use CruiseControl interface:
> ant -Dtest.suites="hdk,stress" install setup run
>
> With best regards, Alexei
>
> On 10/6/07, Alexei Fedotov <al...@gmail.com> wrote:
> > Hello all,
> >
> > I want to thank participants of this discussion for their interest.
> > Since the trunk is unfrozen I also want to revive a discussion and get
> > someone interested in committing the changes [1].
> >
> > BTW, do you think it might be feasible to include p-unit as an
> > external repository by modifying svn:externals property at
> > http://svn.apache.org/repos/asf/harmony/enhanced/buildtest/branches/2.0/tests/tools/
> > instead of fetching it from my build script?
> >
> > Thanks!
> >
> > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> >
> >
> > On 10/2/07, Alexei Zakharov <al...@gmail.com> wrote:
> > > +1
> > >
> > > 2007/10/2, Stepan Mishura <st...@gmail.com>:
> > > > On 10/2/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > Hello,
> > > > >
> > > > > Could anyone be so kind to review my changes to the test harness [1]?
> > > > > I believe they are pretty harmless.
> > > > >
> > > >
> > > > I think we shouldn't hurry with the update for the harness during code
> > > > freeze - there are several suites that depend on the harness. IMO
> > > > should be done after M3 release.
> > > >
> > > > Thanks,
> > > > Stepan.
> > > >
> > > > > Thank you in advance!
> > > > >
> > > > > [1] http://issues.apache.org/jira/secure/attachment/12366764/harness_test_handler.patch
> > > > >
> > > > > On 10/1/07, Andrew Zhang <zh...@gmail.com> wrote:
> > > > > > On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > > >
> > > > > > > Hello,
> > > > > > >
> > > > > > > The patch for the stress test suite which make use of p-unit
> > > > > > > performace infrastructure for comparative stress load analysis is
> > > > > > > ready  [1]. I want to ask a commuity when it might be committed.
> > > > > >
> > > > > >
> > > > > > Hi Alexei,
> > > > > >
> > > > > > Great work!  Super thanks to YOU!
> > > > > >
> > > > > > Thanks, Alexei
> > > > > > >
> > > > > > > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> > > > > > >
> > > > > > > P.S. Special thanks to Andrew Zhang for his support.
> > > > > > >
> > > > > > > On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > > > > Alexey,
> > > > > > > > Thanks for reference!
> > > > > > > [...]
> > > > > > > --
> > > > > > > With best regards,
> > > > > > > Alexei,
> > > > > > > ESSD, Intel
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Best regards,
> > > > > > Andrew Zhang
> > > > > >
> > > > > > http://zhanghuangzhu.blogspot.com/
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > > With best regards,
> > > > > Alexei,
> > > > > ESSD, Intel
> > > > >
> > > >
> > >
> > >
> > > --
> > > Alexei Zakharov,
> > > Intel ESSD
> > >
> >
> >
> > --
> > With best regards,
> > Alexei,
> > ESSD, Intel
> >
>
>
> --
> With best regards,
> Alexei,
> ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Fedotov <al...@gmail.com>.
Dear committers,

Please excuse me for being persistent. I continue updating this thread
to keep it at the top of your mailboxes since I want one of you to get
a honorable responsibility of supporting a stress test suite.

To check changes one my use CruiseControl interface:
ant -Dtest.suites="hdk,stress" install setup run

With best regards, Alexei

On 10/6/07, Alexei Fedotov <al...@gmail.com> wrote:
> Hello all,
>
> I want to thank participants of this discussion for their interest.
> Since the trunk is unfrozen I also want to revive a discussion and get
> someone interested in committing the changes [1].
>
> BTW, do you think it might be feasible to include p-unit as an
> external repository by modifying svn:externals property at
> http://svn.apache.org/repos/asf/harmony/enhanced/buildtest/branches/2.0/tests/tools/
> instead of fetching it from my build script?
>
> Thanks!
>
> [1] http://issues.apache.org/jira/browse/HARMONY-4114
>
>
> On 10/2/07, Alexei Zakharov <al...@gmail.com> wrote:
> > +1
> >
> > 2007/10/2, Stepan Mishura <st...@gmail.com>:
> > > On 10/2/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > Hello,
> > > >
> > > > Could anyone be so kind to review my changes to the test harness [1]?
> > > > I believe they are pretty harmless.
> > > >
> > >
> > > I think we shouldn't hurry with the update for the harness during code
> > > freeze - there are several suites that depend on the harness. IMO
> > > should be done after M3 release.
> > >
> > > Thanks,
> > > Stepan.
> > >
> > > > Thank you in advance!
> > > >
> > > > [1] http://issues.apache.org/jira/secure/attachment/12366764/harness_test_handler.patch
> > > >
> > > > On 10/1/07, Andrew Zhang <zh...@gmail.com> wrote:
> > > > > On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > >
> > > > > > Hello,
> > > > > >
> > > > > > The patch for the stress test suite which make use of p-unit
> > > > > > performace infrastructure for comparative stress load analysis is
> > > > > > ready  [1]. I want to ask a commuity when it might be committed.
> > > > >
> > > > >
> > > > > Hi Alexei,
> > > > >
> > > > > Great work!  Super thanks to YOU!
> > > > >
> > > > > Thanks, Alexei
> > > > > >
> > > > > > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> > > > > >
> > > > > > P.S. Special thanks to Andrew Zhang for his support.
> > > > > >
> > > > > > On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > > > Alexey,
> > > > > > > Thanks for reference!
> > > > > > [...]
> > > > > > --
> > > > > > With best regards,
> > > > > > Alexei,
> > > > > > ESSD, Intel
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Best regards,
> > > > > Andrew Zhang
> > > > >
> > > > > http://zhanghuangzhu.blogspot.com/
> > > > >
> > > >
> > > >
> > > > --
> > > > With best regards,
> > > > Alexei,
> > > > ESSD, Intel
> > > >
> > >
> >
> >
> > --
> > Alexei Zakharov,
> > Intel ESSD
> >
>
>
> --
> With best regards,
> Alexei,
> ESSD, Intel
>


-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Fedotov <al...@gmail.com>.
Hello all,

I want to thank participants of this discussion for their interest.
Since the trunk is unfrozen I also want to revive a discussion and get
someone interested in committing the changes [1].

BTW, do you think it might be feasible to include p-unit as an
external repository by modifying svn:externals property at
http://svn.apache.org/repos/asf/harmony/enhanced/buildtest/branches/2.0/tests/tools/
instead of fetching it from my build script?

Thanks!

[1] http://issues.apache.org/jira/browse/HARMONY-4114


On 10/2/07, Alexei Zakharov <al...@gmail.com> wrote:
> +1
>
> 2007/10/2, Stepan Mishura <st...@gmail.com>:
> > On 10/2/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > Hello,
> > >
> > > Could anyone be so kind to review my changes to the test harness [1]?
> > > I believe they are pretty harmless.
> > >
> >
> > I think we shouldn't hurry with the update for the harness during code
> > freeze - there are several suites that depend on the harness. IMO
> > should be done after M3 release.
> >
> > Thanks,
> > Stepan.
> >
> > > Thank you in advance!
> > >
> > > [1] http://issues.apache.org/jira/secure/attachment/12366764/harness_test_handler.patch
> > >
> > > On 10/1/07, Andrew Zhang <zh...@gmail.com> wrote:
> > > > On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > >
> > > > > Hello,
> > > > >
> > > > > The patch for the stress test suite which make use of p-unit
> > > > > performace infrastructure for comparative stress load analysis is
> > > > > ready  [1]. I want to ask a commuity when it might be committed.
> > > >
> > > >
> > > > Hi Alexei,
> > > >
> > > > Great work!  Super thanks to YOU!
> > > >
> > > > Thanks, Alexei
> > > > >
> > > > > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> > > > >
> > > > > P.S. Special thanks to Andrew Zhang for his support.
> > > > >
> > > > > On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > > Alexey,
> > > > > > Thanks for reference!
> > > > > [...]
> > > > > --
> > > > > With best regards,
> > > > > Alexei,
> > > > > ESSD, Intel
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Best regards,
> > > > Andrew Zhang
> > > >
> > > > http://zhanghuangzhu.blogspot.com/
> > > >
> > >
> > >
> > > --
> > > With best regards,
> > > Alexei,
> > > ESSD, Intel
> > >
> >
>
>
> --
> Alexei Zakharov,
> Intel ESSD
>


-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Zakharov <al...@gmail.com>.
+1

2007/10/2, Stepan Mishura <st...@gmail.com>:
> On 10/2/07, Alexei Fedotov <al...@gmail.com> wrote:
> > Hello,
> >
> > Could anyone be so kind to review my changes to the test harness [1]?
> > I believe they are pretty harmless.
> >
>
> I think we shouldn't hurry with the update for the harness during code
> freeze - there are several suites that depend on the harness. IMO
> should be done after M3 release.
>
> Thanks,
> Stepan.
>
> > Thank you in advance!
> >
> > [1] http://issues.apache.org/jira/secure/attachment/12366764/harness_test_handler.patch
> >
> > On 10/1/07, Andrew Zhang <zh...@gmail.com> wrote:
> > > On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > >
> > > > Hello,
> > > >
> > > > The patch for the stress test suite which make use of p-unit
> > > > performace infrastructure for comparative stress load analysis is
> > > > ready  [1]. I want to ask a commuity when it might be committed.
> > >
> > >
> > > Hi Alexei,
> > >
> > > Great work!  Super thanks to YOU!
> > >
> > > Thanks, Alexei
> > > >
> > > > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> > > >
> > > > P.S. Special thanks to Andrew Zhang for his support.
> > > >
> > > > On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > > Alexey,
> > > > > Thanks for reference!
> > > > [...]
> > > > --
> > > > With best regards,
> > > > Alexei,
> > > > ESSD, Intel
> > > >
> > >
> > >
> > >
> > > --
> > > Best regards,
> > > Andrew Zhang
> > >
> > > http://zhanghuangzhu.blogspot.com/
> > >
> >
> >
> > --
> > With best regards,
> > Alexei,
> > ESSD, Intel
> >
>


-- 
Alexei Zakharov,
Intel ESSD

Re: [bti][p-unit] finally, it is ready

Posted by Stepan Mishura <st...@gmail.com>.
On 10/2/07, Alexei Fedotov <al...@gmail.com> wrote:
> Hello,
>
> Could anyone be so kind to review my changes to the test harness [1]?
> I believe they are pretty harmless.
>

I think we shouldn't hurry with the update for the harness during code
freeze - there are several suites that depend on the harness. IMO
should be done after M3 release.

Thanks,
Stepan.

> Thank you in advance!
>
> [1] http://issues.apache.org/jira/secure/attachment/12366764/harness_test_handler.patch
>
> On 10/1/07, Andrew Zhang <zh...@gmail.com> wrote:
> > On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
> > >
> > > Hello,
> > >
> > > The patch for the stress test suite which make use of p-unit
> > > performace infrastructure for comparative stress load analysis is
> > > ready  [1]. I want to ask a commuity when it might be committed.
> >
> >
> > Hi Alexei,
> >
> > Great work!  Super thanks to YOU!
> >
> > Thanks, Alexei
> > >
> > > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> > >
> > > P.S. Special thanks to Andrew Zhang for his support.
> > >
> > > On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > > Alexey,
> > > > Thanks for reference!
> > > [...]
> > > --
> > > With best regards,
> > > Alexei,
> > > ESSD, Intel
> > >
> >
> >
> >
> > --
> > Best regards,
> > Andrew Zhang
> >
> > http://zhanghuangzhu.blogspot.com/
> >
>
>
> --
> With best regards,
> Alexei,
> ESSD, Intel
>

Re: [bti][p-unit] finally, it is ready

Posted by Alexei Fedotov <al...@gmail.com>.
Hello,

Could anyone be so kind to review my changes to the test harness [1]?
I believe they are pretty harmless.

Thank you in advance!

[1] http://issues.apache.org/jira/secure/attachment/12366764/harness_test_handler.patch

On 10/1/07, Andrew Zhang <zh...@gmail.com> wrote:
> On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
> >
> > Hello,
> >
> > The patch for the stress test suite which make use of p-unit
> > performace infrastructure for comparative stress load analysis is
> > ready  [1]. I want to ask a commuity when it might be committed.
>
>
> Hi Alexei,
>
> Great work!  Super thanks to YOU!
>
> Thanks, Alexei
> >
> > [1] http://issues.apache.org/jira/browse/HARMONY-4114
> >
> > P.S. Special thanks to Andrew Zhang for his support.
> >
> > On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > > Alexey,
> > > Thanks for reference!
> > [...]
> > --
> > With best regards,
> > Alexei,
> > ESSD, Intel
> >
>
>
>
> --
> Best regards,
> Andrew Zhang
>
> http://zhanghuangzhu.blogspot.com/
>


-- 
With best regards,
Alexei,
ESSD, Intel

Re: [bti][p-unit] finally, it is ready

Posted by Andrew Zhang <zh...@gmail.com>.
On 10/1/07, Alexei Fedotov <al...@gmail.com> wrote:
>
> Hello,
>
> The patch for the stress test suite which make use of p-unit
> performace infrastructure for comparative stress load analysis is
> ready  [1]. I want to ask a commuity when it might be committed.


Hi Alexei,

Great work!  Super thanks to YOU!

Thanks, Alexei
>
> [1] http://issues.apache.org/jira/browse/HARMONY-4114
>
> P.S. Special thanks to Andrew Zhang for his support.
>
> On 7/10/07, Alexei Fedotov <al...@gmail.com> wrote:
> > Alexey,
> > Thanks for reference!
> [...]
> --
> With best regards,
> Alexei,
> ESSD, Intel
>



-- 
Best regards,
Andrew Zhang

http://zhanghuangzhu.blogspot.com/