> On 2012-03-16 14:28:24, Chris Mattmann wrote: > > trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java, line > > 568 > > <https://reviews.apache.org/r/4375/diff/2/?file=93048#file93048line568> > > > > I like this having an explicit step now.
after this patch goes in, the trunk CAS-PGE will have a feature which was attempted in wengine branch CAS-PGE but i never got working right... when multiple CAS-PGEs run in the same JVM, which is the normal case since several will end up in a CAS-Resource's BatchStub, they will log on top of each other since the logger was based on the class name and CAS-PGE usually had the same class name so was impossible to filter loggers by that... so it was really hard to sift through the logs finding relevant logger for a given CAS-PGE instance... this patch fixed that by making logging base on PGETask/Name (or legacy PGETask_Name) - brian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4375/#review6035 ----------------------------------------------------------- On 2012-03-16 08:25:17, brian Foster wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4375/ > ----------------------------------------------------------- > > (Updated 2012-03-16 08:25:17) > > > Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and > Thomas Bennett. > > > Summary > ------- > > This handler will allow for multiple CAS-PGE instances to exist in the same > JVM while routing the logging from each to their own log files in their > execution directory > > ----- > This patch also includes: > - adding support for PgeMetadata to take PgeTaskMetKeys directly > - some cleanup and reorganizing of PgeTaskInstance > > > This addresses bug OODT-414. > https://issues.apache.org/jira/browse/OODT-414 > > > Diffs > ----- > > trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java > 1301267 > trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogHandler.java > PRE-CREATION > trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogRecord.java > PRE-CREATION > trunk/pge/src/main/java/org/apache/oodt/cas/pge/metadata/PgeMetadata.java > 1300798 > > trunk/pge/src/main/java/org/apache/oodt/cas/pge/metadata/PgeTaskMetKeys.java > 1301267 > trunk/pge/src/test/org/apache/oodt/cas/pge/TestPGETaskInstance.java 1301267 > trunk/pge/src/test/org/apache/oodt/cas/pge/logging/TestPgeLogHandler.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/4375/diff > > > Testing > ------- > > Added and updated unit-tests > > > Thanks, > > brian > >
