Skip to content

Commit 6408864

Browse files
committed
workload/tpcds: remove remaining queries that can't be parsed
This commit removes some queries from `QueriesByNumber` that can't be parsed because of the `rollup()` function. Epic: None Release note: None
1 parent f09c7b8 commit 6408864

File tree

1 file changed

+27
-13
lines changed

1 file changed

+27
-13
lines changed

pkg/workload/tpcds/queries.go

Lines changed: 27 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,12 @@ const NumQueries = 99
1111
// QueriesByNumber is a mapping from the number of a TPC-DS query to the actual
1212
// query. Only queries that can be parsed by CockroachDB are present.
1313
var QueriesByNumber = map[int]string{
14-
1: query1,
15-
2: query2,
16-
3: query3,
17-
4: query4,
18-
5: query5,
14+
1: query1,
15+
2: query2,
16+
3: query3,
17+
4: query4,
18+
// TODO: adjust this query.
19+
//5: query5,
1920
6: query6,
2021
7: query7,
2122
8: query8,
@@ -24,15 +25,18 @@ var QueriesByNumber = map[int]string{
2425
11: query11,
2526
12: query12,
2627
13: query13,
27-
14: query14,
28+
// TODO: adjust this query.
29+
// 14: query14,
2830
15: query15,
2931
16: query16,
3032
17: query17,
31-
18: query18,
33+
// TODO: adjust this query.
34+
// 18: query18,
3235
19: query19,
3336
20: query20,
3437
21: query21,
35-
22: query22,
38+
// TODO: adjust the query.
39+
// 22: query22,
3640
23: query23,
3741
24: query24,
3842
25: query25,
@@ -79,7 +83,8 @@ var QueriesByNumber = map[int]string{
7983
64: query64,
8084
65: query65,
8185
66: query66,
82-
67: query67,
86+
// TODO: adjust the query.
87+
// 67: query67,
8388
68: query68,
8489
69: query69,
8590
// TODO(yuzefovich): adjust the query.
@@ -90,10 +95,12 @@ var QueriesByNumber = map[int]string{
9095
74: query74,
9196
75: query75,
9297
76: query76,
93-
77: query77,
98+
// TODO: adjust the query.
99+
// 77: query77,
94100
78: query78,
95101
79: query79,
96-
80: query80,
102+
// TODO: adjust the query.
103+
// 80: query80,
97104
81: query81,
98105
82: query82,
99106
83: query83,
@@ -118,9 +125,16 @@ var QueriesByNumber = map[int]string{
118125

119126
// TODO(yuzefovich): remove once these queries are "enabled."
120127
var (
128+
_ = query5
129+
_ = query14
130+
_ = query18
131+
_ = query22
121132
_ = query27
122133
_ = query36
134+
_ = query67
123135
_ = query70
136+
_ = query77
137+
_ = query80
124138
_ = query86
125139
)
126140

@@ -2985,7 +2999,7 @@ SELECT
29852999
c_birth_country,
29863000
c_login,
29873001
c_email_address,
2988-
c_last_review_date_sk,
3002+
c_last_review_date,
29893003
ctr_total_return
29903004
FROM
29913005
customer_total_return AS ctr1,
@@ -3016,7 +3030,7 @@ ORDER BY
30163030
c_birth_country,
30173031
c_login,
30183032
c_email_address,
3019-
c_last_review_date_sk,
3033+
c_last_review_date,
30203034
ctr_total_return
30213035
LIMIT
30223036
100;

0 commit comments

Comments
 (0)