On Sat, Jul 30, 2011 at 09:11:32AM +0000, Pierre Casenove wrote: > Crapy hotmail...Please find attached the diff.I've created BZ 724963 also.
> diff config_queries.xml config_queries.xml.orig > 85,90c85,90 > < FROM rhnConfigFile CF > < LEFT OUTER JOIN rhnConfigRevision CR ON (CR.config_file_id = CF.id) > < LEFT OUTER JOIN rhnSnapshotConfigRevision SCR ON (SCR.config_revision_id > = CR.id) > < LEFT OUTER JOIN rhnConfigFileName CFN ON (CFN.id = CF.config_file_name_id) > < LEFT OUTER JOIN rhnConfigContent CCon ON (CCon.id = CR.config_content_id) > < LEFT OUTER JOIN rhnChecksumView Csum ON (Csum.id = CCon.checksum_id) > --- > > FROM rhnConfigContent CCon, > > rhnConfigFileName CFN, > > rhnConfigFile CF, > > rhnConfigRevision CR, > > rhnSnapshotConfigRevision SCR, > > rhnChecksumView Csum > 91a92,96 > > AND SCR.config_revision_id = CR.id > > AND CR.config_content_id = CCon.id (+) > > AND CR.config_file_id = CF.id > > AND CF.config_file_name_id = CFN.id > > AND CCon.checksum_id = Csum.id (+) This is incorrect. There are two outer joins in the original, and five in your version. You mustn't just replace every join with outer join -- the semantics is quite different. -- Jan Pazdziora Principal Software Engineer, Satellite Engineering, Red Hat _______________________________________________ Spacewalk-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/spacewalk-list
