Good idea, I'll go ahead and file a bug to track this.

I think it is a much bigger issue. In my opinion, we need to start from scratch, there is a little code to save but the rest should go. I would also like to consolidate the perf testing for Beehive in BEEHIVE_HOME/test/perf, to reduce management and infrastructure overhead. My preference would be to replace TCH with a JUnit system.

Patrick.

Steven Tocco wrote:

Should we file a CR to this effect and mark for TBD?

I suspect we should.  Only question is it against the scripts or a
bigger issue?

Thanks
Steve

-----Original Message-----
From: Patrick Osborne Sent: Tuesday, April 26, 2005 10:59 AM
To: Eddie O'Neil
Cc: Beehive Developers
Subject: Re: controls perf test check-in request



No problem Eddie. If possible, I would like to get this patch in by 5pm

tonight when the automated tests are run again. James let me know if you have questions.
I'm not adding scripts, just editing them for the proper classpath. I agree that shell scripts shouldn't be used, but this is what I inherited


from Jamie.  I'm just trying to keep the current system running until we

have time to build a better one.

Cheers,
Patrick.

Eddie O'Neil wrote:



Patrick--

Hey; I'd rather have this go through one of the people focused on controls testing. James, can you look at this patch?

One question I do have about the patch is why we're adding shell scripts to the tree? Why aren't these just <java> calls in an Ant build file? Seems easier to maintain / test across platforms and


shells.


Thanks!

Eddie



Patrick Osborne wrote:



Hi Eddie,

Would you please check-in the following controls perf test build changes. This will fix several broken tests.

Thanks,
Patrick.





------------------------------------------------------------------------


Index: controls/test/perf/bin/perf.bat
===================================================================
--- controls/test/perf/bin/perf.bat (revision 164636)
+++ controls/test/perf/bin/perf.bat (working copy)
@@ -1 +1 @@
-java -Xmx64m -classpath




%BEEHIVE_PERFROOT%\build;%BEEHIVE_PERFROOT%\ctlbuild;%BEEHIVE_PERFROOT%\
..\..\..\controls\build\jars\controls.jar


org.apache.beehive.controls.perf.standalone.%* +java -Xmx64m -classpath



%BEEHIVE_PERFROOT%\build;%BEEHIVE_PERFROOT%\ctlbuild;%BEEHIVE_PERFROOT%\
..\..\..\build\dist\apache-beehive-incubating-svn-snapshot\lib\controls\
beehive-controls.jar


org.apache.beehive.controls.perf.standalone.%* Index: controls/test/perf/bin/perf.sh
===================================================================
--- controls/test/perf/bin/perf.sh (revision 164636)
+++ controls/test/perf/bin/perf.sh (working copy)
@@ -1 +1 @@
-java -Xmx64m -classpath




$BEEHIVE_PERFROOT/build:$BEEHIVE_PERFROOT/ctlbuild:$BEEHIVE_PERFROOT/../
../../controls/build/jars/controls.jar


org.apache.beehive.controls.perf.standalone.$* +java -Xmx64m -classpath



$BEEHIVE_PERFROOT/build:$BEEHIVE_PERFROOT/ctlbuild:$BEEHIVE_PERFROOT/../
../../build/dist/apache-beehive-incubating-svn-snapshot/lib/controls/bee
hive-controls.jar


org.apache.beehive.controls.perf.standalone.$* Index: controls/test/perf/build.xml
===================================================================
--- controls/test/perf/build.xml (revision 164636)
+++ controls/test/perf/build.xml (working copy)
@@ -205,8 +205,10 @@
<property name="gtlf.dtd.use.remote" value="true"/>
<property name="gtlf.dtd.remote"




value="http://incubator.apache.org/beehive/dtd/gtlf/gtlf-config-2.0.dtd";
/>


<property name="gtlf.dtd.local" value="infra/gtlf/gtlf-config-2.0.dtd"/>
- <property name="tch.log.basedir"


value="${control.perf.logs}"/>


+ <property name="tch.log.basedir" location="${control.perf.logs}"/>
<property name="tch.base-logfile-name" value="tch"/>
+ <echo>tch.log.basedir( ${tch.log.basedir} )</echo> + <mkdir dir="${tch.log.basedir}"/>
<tch tchHome="${tch.home}"
testfile="${test-suite}"
fork="true"
@@ -233,6 +235,7 @@
</target> <target name="checkin.tests.client">
+ <echo>Running test client</echo>
<property name="test-suite" value="${control.perf.root}/cases/controls-perf.xml"/>
<property name="test.freq" value="bvt"/>
<property name="tch.log.debug-level" value="0"/>









Reply via email to