db-jdo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael Watzek <mwa.t...@spree.de>
Subject Re: 2nd patch implementing logging proposal
Date Tue, 23 Aug 2005 15:40:44 GMT
Hi Michael,

please find the attached patch for review implementing a summary of the 
TCK results for all configurations. In addition the patch implements a 
final result message saying how many configurations passed/failed.

For this purpose, I decided to make static inner class 
"ConsoleFileOutput" an outer class, as this class is now used by class 
"BatchResultPrinter", too.

A good start for a review is method "BatchResultPrinter.printFooter". 
This method is called once for each TCK configuration. There, a final 
result message is printed to a summary file for each configuration. 
Moreover, that method deserializes/serializes a result instance 
containing the number of passed/failed configurations. After all 
configurations have been run, that instance is deserialized for a final 
message of passed/failed configurations. Afterwards, the file storing 
that instance is deleted.

I attached a result summary file created today as an example.


> Hi Michael,
> thanks for the patch. I agree with all your changes and checked in the
> patch as you sent it over.
> BTW, are you planning to look into the remaining part about adding a new
> file to the logs directory including a summary of the runtck results for
> all the configurations?
> Regards Michael
>>Hi Michael,
>>please find the attached patch implementing the 2nd version of the
>>logging proposal for review. The changes to the first implementation
>>version are:
>>- project.properties
>>-- Property "jdo.tck.util.enhancer.sources" has been removed.
>>- maven.xml
>>-- All log directories are created by maven. Before, only the database
>>log directory was created by maven.
>>-- The enhancer goal "enhance.prepare" specifies prereqs="java:compile,
>>-- The system property "no.log.file" is not passed to the enhancer VM
>>any more.
>>-- Property "jdo.tck.util.enhancer.sources" is not passed to the java
>>compiler in goal "jdo.tck.util.enhancer.sources" any more.
>>- BatcherTestRunner.java
>>-- The static initiallizerhas been removed.
>>-- An instance of "ConsoleFileOutput" is passed to the constructor call
>>of class "BatchResultPrinter". Before, "System.out" was passed.
>>-- Log Directories are not created by this class any more.
>>-- Methods "changeFileNameCreateDirectory have been renamed to
>>-- Methods "getSystemPropertyAsPartialFileName" have been removed.
>>-- Java doc comments have been added.
>>- TCKFileFileAppender
>>-- Java doc comments have been added.
>>Michael Watzek                  Tech@Spree Engineering GmbH
>>mailto:mwa.tech@spree.de        Buelowstr. 66
>>Tel.:  ++49/30/235 520 36       10783 Berlin - Germany
>>Fax.:  ++49/30/217 520 12       http://www.spree.de/

Michael Watzek                  Tech@Spree Engineering GmbH
mailto:mwa.tech@spree.de        Buelowstr. 66
Tel.:  ++49/30/235 520 36       10783 Berlin - Germany
Fax.:  ++49/30/217 520 12       http://www.spree.de/

View raw message