4 Comments

1. 

   Peter P on 2025-04-09 at 14:59

   I’m hesitant to reply – because i have a feeling i haven’t seen the catch.
   If changing the view is the only option and minimizing cpu is the goal – this is my suggestion – and a bit more readable then the original
   Reduces cpu time by approx half

   CREATE VIEW [dbo].[exam_grades]
   AS
   SELECT ssg.s_id
   ,s.s_name
   ,ssg.exam_date
   ,best_grade = LEAST(ssg.mathematics, ssg.english, ssg.biology, ssg.geographics, ssg.philosophy, ssg.sports)
   ,worst_grade = GREATEST(ssg.mathematics, ssg.english, ssg.biology, ssg.geographics, ssg.philosophy, ssg.sports)
   FROM dbo.student_school_grades ssg
   JOIN dbo.students s
   ON s.s_id = ssg.s_id;

   Using an indexed view will perform better – but is not my preferred weapon of choice:
   Reduces cpu time to 1/4

   CREATE VIEW [dbo].[exam_grades] WITH SCHEMABINDING
   AS
   SELECT ssg.s_id
   ,s.s_name
   ,ssg.exam_date
   ,best_grade = LEAST(ssg.mathematics, ssg.english, ssg.biology, ssg.geographics, ssg.philosophy, ssg.sports)
   ,worst_grade = GREATEST(ssg.mathematics, ssg.english, ssg.biology, ssg.geographics, ssg.philosophy, ssg.sports)
   FROM dbo.student_school_grades ssg
   JOIN dbo.students s
   ON s.s_id = ssg.s_id;

   CREATE UNIQUE CLUSTERED INDEX someviewname ON [dbo].[exam_grades3] (s_id, exam_date)

   Reply
   • 

     Uwe Ricken on 2025-04-12 at 12:43

     Dear Peter,

     thank you so much for solving the sqlchallenge.
     To be honest – I didn’t had your solution in my mind but it is absolutely great.

     I will give it a try when I write about the solutions.

     Reply
2. 

   Gabriel Köhl on 2025-04-12 at 10:18

   Joa,

   also Peter hat ja schon den Königsweg gezeigt. Ich habe somit weiter versucht, die Werte einzeln wie im Original zu behandeln, da die View schon sehr weh tat im Auge. Ich habe die Auswertung auf ein gesamtheitliche Zeile ausgewertet und nicht mehr jeden Wert gegen jeden Wert. Das ganze in einen CTE. Die eigentliche Abfrage war somit bei 42ms CPU. Dann kam aber der Join und wir waren wieder bei 110ms. Also statt CTE einen TempTable genommen, damit wir einen statischen Datensatz haben, der schneller gejoined werden kann. Hatt funktioniert, immerhin 80ms im Schnit. Aber die Aufgabe ist trotzdem failed, da ich in einer View keine #Table maintainen kann 😉 Aber ich wollts trotzdem mal zeigen

   SET STATISTICS TIME ON;
   DROP TABLE IF EXISTS #GradeData;
   SELECT
   [s_id],
   [exam_date],
   (
   SELECT
   MIN(grade)
   FROM
   (VALUES
   (COALESCE([mathematics], ‘F’)),
   (COALESCE([english], ‘F’)),
   (COALESCE([biology], ‘F’)),
   (COALESCE([geographics], ‘F’)),
   (COALESCE([philosophy], ‘F’)),
   (COALESCE([sports], ‘F’))
   ) AS Grades(grade)
   ) AS [best_grade],
   (
   SELECT
   MAX(grade)
   FROM
   (VALUES
   (COALESCE([mathematics], ‘F’)),
   (COALESCE([english], ‘F’)),
   (COALESCE([biology], ‘F’)),
   (COALESCE([geographics], ‘F’)),
   (COALESCE([philosophy], ‘F’)),
   (COALESCE([sports], ‘F’))
   ) AS Grades(grade)
   ) AS [worst_grade]
   INTO
   #GradeData
   FROM
   [88FA498A-C65E-4BFD-8C79-B5E9F81DE10B].[dbo].[student_school_grades];

   SELECT
   st.s_name,
   gd.exam_date,
   gd.best_grade,
   gd.worst_grade
   FROM
   #GradeData as gd
   JOIN
   [88FA498A-C65E-4BFD-8C79-B5E9F81DE10B].[dbo].[students] as st ON gd.s_id = st.s_id;

   DROP TABLE IF EXISTS #GradeData;

   Dann habe ich noch mal ein bisschen gegoogled.

   https://www.brentozar.com/archive/2022/12/should-you-use-sql-server-2022s-greatest-and-least/

   Und es ist eigentlich wie immer, diese Neuerungen sind größtenteils QoL Changes aber nicht performanter. Zumal bei Brent auch noch Indexe und Zahlen genutzt werden. Aber ich habe dann noch mal nachgedacht und Funktionen in in Queries sind eh immer so kleiner Killer. Also mal richtig OldSchool Sql probiert und siehe da, 40-60 ms CPU Zeit und dazu in einer View nutzbar 🙂

   SET STATISTICS TIME ON;

   SELECT
   st.s_name,
   ssg.exam_date,
   CASE
   WHEN ‘A’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘A’
   WHEN ‘B’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘B’
   WHEN ‘C’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘C’
   WHEN ‘D’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘D’
   WHEN ‘E’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘E’
   ELSE ‘F’
   END AS best_grade,

   CASE
   WHEN ‘F’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘F’
   WHEN ‘E’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘E’
   WHEN ‘D’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘D’
   WHEN ‘C’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘C’
   WHEN ‘B’ IN (ssg.[mathematics], ssg.[english], ssg.[biology], ssg.[geographics], ssg.[philosophy], ssg.[sports]) THEN ‘B’
   ELSE ‘A’
   END AS worst_grade
   FROM
   [88FA498A-C65E-4BFD-8C79-B5E9F81DE10B].[dbo].[student_school_grades] as ssg
   JOIN
   [88FA498A-C65E-4BFD-8C79-B5E9F81DE10B].[dbo].[students] as st ON ssg.s_id = st.s_id

   Hat Spaß gemahct und gelernt 🙂

   Reply
   • 

     Uwe Ricken on 2025-04-12 at 12:42

     Vielen Dank, lieber Daniel.
     Deine Lösung ist sehr interessant.

     Reply