[Spacewalk-list] Error 500 on snapshots pages

Jan Pazdziora jpazdziora at redhat.com
Thu Aug 4 13:48:25 UTC 2011


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




More information about the Spacewalk-list mailing list