[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