Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-22161] [SQL] Add Impala-modified TPC-DS queries #19386

Closed
wants to merge 3 commits into from

Conversation

gatorsmile
Copy link
Member

What changes were proposed in this pull request?

Added IMPALA-modified TPCDS queries to TPC-DS query suites.

How was this patch tested?

N/A

@gatorsmile
Copy link
Member Author

@SparkQA
Copy link

SparkQA commented Sep 29, 2017

Test build #82294 has finished for PR 19386 at commit 0980974.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • and i_class in ('personal', 'portable', 'reference', 'self-help')
  • and i_class in ('accessories', 'classical', 'fragrances', 'pants')
  • and i_class in ('personal','portable','reference','self-help')
  • and i_class in ('accessories','classical','fragrances','pants')
  • and i_class in ('wallpaper', 'parenting', 'musical'))
  • and i_class in ('womens', 'birdal', 'pants')))

@SparkQA
Copy link

SparkQA commented Sep 29, 2017

Test build #82291 has finished for PR 19386 at commit 0dd6b47.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • and i_class in ('personal', 'portable', 'reference', 'self-help')
  • and i_class in ('accessories', 'classical', 'fragrances', 'pants')
  • and i_class in ('personal','portable','reference','self-help')
  • and i_class in ('accessories','classical','fragrances','pants')
  • and i_class in ('wallpaper', 'parenting', 'musical'))
  • and i_class in ('womens', 'birdal', 'pants')))

@SparkQA
Copy link

SparkQA commented Sep 29, 2017

Test build #82292 has finished for PR 19386 at commit 5cfa6d7.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 29, 2017

Test build #82298 has finished for PR 19386 at commit 96764d6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • and i_class in ('personal', 'portable', 'reference', 'self-help')
  • and i_class in ('accessories', 'classical', 'fragrances', 'pants')
  • and i_class in ('personal','portable','reference','self-help')
  • and i_class in ('accessories','classical','fragrances','pants')
  • and i_class in ('wallpaper', 'parenting', 'musical'))
  • and i_class in ('womens', 'birdal', 'pants')))

@liancheng
Copy link
Contributor

LGTM

1 similar comment
@juliuszsompolski
Copy link
Contributor

LGTM

@gatorsmile
Copy link
Member Author

Thanks! Merged to master/2.2

asfgit pushed a commit that referenced this pull request Sep 29, 2017
## What changes were proposed in this pull request?

Added IMPALA-modified TPCDS queries to TPC-DS query suites.

- Ref: https://github.com/cloudera/impala-tpcds-kit/tree/master/queries

## How was this patch tested?
N/A

Author: gatorsmile <gatorsmile@gmail.com>

Closes #19386 from gatorsmile/addImpalaQueries.

(cherry picked from commit 9ed7394)
Signed-off-by: gatorsmile <gatorsmile@gmail.com>
@asfgit asfgit closed this in 9ed7394 Sep 29, 2017
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
## What changes were proposed in this pull request?

Added IMPALA-modified TPCDS queries to TPC-DS query suites.

- Ref: https://github.com/cloudera/impala-tpcds-kit/tree/master/queries

## How was this patch tested?
N/A

Author: gatorsmile <gatorsmile@gmail.com>

Closes apache#19386 from gatorsmile/addImpalaQueries.

(cherry picked from commit 9ed7394)
Signed-off-by: gatorsmile <gatorsmile@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants