Skip to content

Commit

Permalink
sql: Fix TPCH workload
Browse files Browse the repository at this point in the history
Fix a couple problems in the TPCH workload:

1. Use r_name = 'EUROPE' in Q2 to match the TPCH spec sample values
2. Add ::float cast in Q11 because CRDB does not implicitly cast when
   multiplying float and int.

Fixes cockroachdb#35418

Release note: None
  • Loading branch information
andy-kimball committed Mar 8, 2019
1 parent 38bb1e7 commit 05eaa3a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions pkg/workload/tpch/queries.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ WHERE
AND s_suppkey = ps_suppkey
AND s_nationkey = n_nationkey
AND n_regionkey = r_regionkey
AND r_name = 'AMERICA'
AND r_name = 'EUROPE'
)
ORDER BY
s_acctbal DESC,
Expand Down Expand Up @@ -356,7 +356,7 @@ LIMIT 20;
query11 = `
SELECT
ps_partkey,
sum(ps_supplycost * ps_availqty) AS value
sum(ps_supplycost * ps_availqty::float) AS value
FROM
partsupp,
supplier,
Expand All @@ -367,9 +367,9 @@ WHERE
AND n_name = 'GERMANY'
GROUP BY
ps_partkey HAVING
sum(ps_supplycost * ps_availqty) > (
sum(ps_supplycost * ps_availqty::float) > (
SELECT
sum(ps_supplycost * ps_availqty) * 0.0001
sum(ps_supplycost * ps_availqty::float) * 0.0001
FROM
partsupp,
supplier,
Expand Down

0 comments on commit 05eaa3a

Please sign in to comment.