Skip to content

[VL] Gluten-it: Rename column c_last_review_date to c_last_review_date_sk#11792

Open
zhztheplayer wants to merge 2 commits intoapache:mainfrom
zhztheplayer:wip-it-last-review-col-name
Open

[VL] Gluten-it: Rename column c_last_review_date to c_last_review_date_sk#11792
zhztheplayer wants to merge 2 commits intoapache:mainfrom
zhztheplayer:wip-it-last-review-col-name

Conversation

@zhztheplayer
Copy link
Member

@zhztheplayer zhztheplayer commented Mar 19, 2026

To comply with TPC-DS v2.7's schema specification:

https://www.tpc.org/tpc_documents_current_versions/pdf/tpc-ds_v2.7.0.pdf

The wrong name is causing inconsistency in some internal tests on Q30.

It's needed to add --data-gen=always the first time when a test is executed to regenerate the tables within new schema after this change.

@github-actions github-actions bot added the TOOLS label Mar 19, 2026
@github-actions
Copy link

Run Gluten Clickhouse CI on x86

1 similar comment
@github-actions
Copy link

Run Gluten Clickhouse CI on x86

@zhztheplayer zhztheplayer marked this pull request as ready for review March 19, 2026 09:53
@zhztheplayer zhztheplayer force-pushed the wip-it-last-review-col-name branch from 67ffd0f to 35f00a9 Compare March 20, 2026 08:35
@github-actions
Copy link

Run Gluten Clickhouse CI on x86

StructField("c_login", StringType),
StructField("c_email_address", StringType),
StructField("c_last_review_date", StringType)
StructField("c_last_review_date_sk", StringType)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IntType?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants