Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20343#discussion_r163019362
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/TPCDSQuerySuite.scala ---
@@ -244,7 +244,7 @@ class TPCDSQuerySuite extends BenchmarkQueryTest {
|`web_company_id` INT, `web_company_name` STRING,
`web_street_number` STRING,
|`web_street_name` STRING, `web_street_type` STRING,
`web_suite_number` STRING,
|`web_city` STRING, `web_county` STRING, `web_state` STRING,
`web_zip` STRING,
- |`web_country` STRING, `web_gmt_offset` STRING,
`web_tax_percentage` DECIMAL(5,2))
+ |`web_country` STRING, `web_gmt_offset` DECIMAL(5,2),
`web_tax_percentage` DECIMAL(5,2))
--- End diff --
I double checked the schema changes made in this PR is consistent with the
TPC-DS doc
http://www.tpc.org/tpc_documents_current_versions/pdf/tpc-ds_v2.1.0.pdf
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
