Great work! Thanks for fixing it. Dan
On 2/14/13 8:09 AM, "Chris Mattmann" <[email protected]> wrote: > >----------------------------------------------------------- >This is an automatically generated e-mail. To reply, visit: >https://reviews.apache.org/r/9438/#review16586 >----------------------------------------------------------- > >Ship it! > > >Ship It! > >- Chris Mattmann > > >On Feb. 14, 2013, 3:50 p.m., Chris Mattmann wrote: >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/9438/ >> ----------------------------------------------------------- >> >> (Updated Feb. 14, 2013, 3:50 p.m.) >> >> >> Review request for oodt. >> >> >> Description >> ------- >> >> Patch augments CDEResult with an ordering of the fields, and then uses >>that and a few other functions to make sure that the result row is >>populated in order, pulling from dyn, and const fields where needed. >> >> >> This addresses bug OODT-564. >> https://issues.apache.org/jira/browse/OODT-564 >> >> >> Diffs >> ----- >> >> >>./trunk/xmlps/src/main/java/org/apache/oodt/xmlps/product/XMLPSProductHan >>dler.java 1444629 >> >>./trunk/xmlps/src/main/java/org/apache/oodt/xmlps/structs/CDEResult.java >>1444629 >> >> Diff: https://reviews.apache.org/r/9438/diff/ >> >> >> Testing >> ------- >> >> none yet. >> >> >> Thanks, >> >> Chris Mattmann >> >> >
