customer table: c_last_review_date vs c_last_review_date_sk

gregrahn / tpcds-kit

TPC-DS benchmark kit with some modifications/fixes

320 stars 205 forks source link

customer table: c_last_review_date vs c_last_review_date_sk #28

Open gregrahn opened 7 years ago

gregrahn commented 7 years ago

The definition of customer in tpcds.sql lists the last column as c_last_review_date char(10) however the TPC-DS spec v2.4 list the last column as c_last_review_date_sk identifier with FK to d_date_sk

The DDL looks incorrect. This column change also impacts query30.tpl

gregrahn commented 7 years ago

Fixed in v2.6.0 in query30, but DDL (tpcds.sql) not updated to match in official release. I've made both the column name and type change in tpcds.sql

gregrahn commented 4 years ago

https://tpcorg.fogbugz.com/default.asp?3037_lq3je9v9