Hey Luca, Appreciate it, saw that and no worries and feel free to catch the same thing for me :) Appreciate the peer review!
Also feel free to use: http://reviews.apache.org/ Many of use on the Apache OODT PMC use it and like using it to review patches and changes. You can request an account with infra@ if you don't have one yet, and you can tie back to JIRA, etc. Happy Thanksgiving! Cheers, Chris On Nov 22, 2012, at 3:47 AM, Cinquini, Luca (3880) wrote: > Thanks Chris, comment added. You are right that I should have mentioned the > issue number in the commit. > thanks for the careful review :) > Luca > > On Nov 21, 2012, at 5:09 PM, Mattmann, Chris A (388J) wrote: > >> Hey Luca, >> >> Ah, perfect. If it was for OODT-522, then if you ref that in your commit >> message that helps to >> tie to the issue that you are fixing. And it helps people like me who peer >> review the commits >> to know that it's for something that you already filed one for :) >> >> +1 to adding a comment on OODT-522 about what you updated. >> >> Thanks! >> >> Cheers, >> Chris >> >> On Nov 21, 2012, at 3:57 PM, Cinquini, Luca (3880) wrote: >> >>> Hi Chris, >>> sure, but I have a question first: this is really a fix on the >>> following issue, which got filed and solved a few weeks ago: >>> >>> https://issues.apache.org/jira/browse/OODT-522 >>> >>> Would it be more appropriate to insert a comment in this Jira issue, >>> bringing its development to a close ? >>> >>> thanks, Luca >>> >>> On Nov 21, 2012, at 10:19 AM, Mattmann, Chris A (388J) wrote: >>> >>>> Hi Luca, >>>> >>>> Can you please file a JIRA issue for this? This was a change I believe >>>> that took some time to track down so having it in a JIRA issue will help >>>> others track and understand how it was fixed. It should also appear in >>>> our change log. >>>> >>>> Thanks! >>>> >>>> Cheers, >>>> Chris >>>> >>>> On Nov 21, 2012, at 2:50 AM, <l...@apache.org> >>>> <l...@apache.org> wrote: >>>> >>>>> Author: luca >>>>> Date: Wed Nov 21 10:50:30 2012 >>>>> New Revision: 1412053 >>>>> >>>>> URL: http://svn.apache.org/viewvc?rev=1412053&view=rev >>>>> Log: >>>>> Removing product references upon metadata update, as they will be added >>>>> again later on. >>>>> >>>>> Modified: >>>>> oodt/trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java >>>>> >>>>> Modified: >>>>> oodt/trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java >>>>> URL: >>>>> http://svn.apache.org/viewvc/oodt/trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java?rev=1412053&r1=1412052&r2=1412053&view=diff >>>>> ============================================================================== >>>>> --- >>>>> oodt/trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java >>>>> (original) >>>>> +++ >>>>> oodt/trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java >>>>> Wed Nov 21 10:50:30 2012 >>>>> @@ -523,6 +523,12 @@ public class MetadataResource extends Cu >>>>> // retrieve existing metadata >>>>> metadata = catalog.getMetadata(product); >>>>> >>>>> + // remove product references (as they will be added later) >>>>> + metadata.removeMetadata("reference_orig"); >>>>> + metadata.removeMetadata("reference_data_store"); >>>>> + metadata.removeMetadata("reference_fileSize"); >>>>> + metadata.removeMetadata("reference_mimeType"); >>>>> + >>>>> // merge new and existing metadata >>>>> metadata.addMetadata(newMetadata); >>>>> >>>>> >>>>> >>>> >>> >> >