Skip to content

[SPARK-5213] [SQL] Remove the duplicated SparkSQLParser #5965

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

[SPARK-5213] [SQL] Remove the duplicated SparkSQLParser #5965

wants to merge 2 commits into from

Conversation

@chenghao-intel
Copy link
Contributor

@chenghao-intel chenghao-intel commented May 7, 2015

This is a follow up of #5827 to remove the additional SparkSQLParser

@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Merged build triggered.

@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Merged build started.

@SparkQA
Copy link

SparkQA commented May 7, 2015

Test build #32075 has started for PR 5965 at commit 6b80a60.

@SparkQA
Copy link

SparkQA commented May 7, 2015

Test build #32075 has finished for PR 5965 at commit 6b80a60.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32075/
Test FAILed.

@chenghao-intel chenghao-intel force-pushed the remove_sparksqlparser branch from 6b80a60 to 509a233 Compare May 7, 2015 11:01
@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Merged build triggered.

@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Merged build started.

@SparkQA
Copy link

SparkQA commented May 7, 2015

Test build #32106 has started for PR 5965 at commit 509a233.

@SparkQA
Copy link

SparkQA commented May 7, 2015

Test build #32106 has finished for PR 5965 at commit 509a233.

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

@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Merged build finished. Test PASSed.

@AmplabJenkins
Copy link

AmplabJenkins commented May 7, 2015

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32106/
Test PASSed.

asfgit pushed a commit that referenced this pull request May 7, 2015
@chenghao-intel @marmbrus
[SPARK-5213] [SQL] Remove the duplicated SparkSQLParser
2b0c423 
This is a follow up of #5827 to remove the additional `SparkSQLParser`

Author: Cheng Hao <[email protected]>

Closes #5965 from chenghao-intel/remove_sparksqlparser and squashes the following commits:

509a233 [Cheng Hao] Remove the HiveQlQueryExecution
a5f9e3b [Cheng Hao] Remove the duplicated SparkSQLParser

(cherry picked from commit 074d75d)
Signed-off-by: Michael Armbrust <[email protected]>
@marmbrus
Copy link
Contributor

marmbrus commented May 7, 2015

Thanks, merged to master and 1.4

@asfgit asfgit closed this in 074d75d May 7, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
@chenghao-intel @jeanlyn
[SPARK-5213] [SQL] Remove the duplicated SparkSQLParser
0cba3c5 
This is a follow up of apache#5827 to remove the additional `SparkSQLParser`

Author: Cheng Hao <[email protected]>

Closes apache#5965 from chenghao-intel/remove_sparksqlparser and squashes the following commits:

509a233 [Cheng Hao] Remove the HiveQlQueryExecution
a5f9e3b [Cheng Hao] Remove the duplicated SparkSQLParser
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
@chenghao-intel @jeanlyn
[SPARK-5213] [SQL] Remove the duplicated SparkSQLParser
eed5235 
This is a follow up of apache#5827 to remove the additional `SparkSQLParser`

Author: Cheng Hao <[email protected]>

Closes apache#5965 from chenghao-intel/remove_sparksqlparser and squashes the following commits:

509a233 [Cheng Hao] Remove the HiveQlQueryExecution
a5f9e3b [Cheng Hao] Remove the duplicated SparkSQLParser
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
@chenghao-intel @nemccarthy
[SPARK-5213] [SQL] Remove the duplicated SparkSQLParser
e9b4afa 
This is a follow up of apache#5827 to remove the additional `SparkSQLParser`

Author: Cheng Hao <[email protected]>

Closes apache#5965 from chenghao-intel/remove_sparksqlparser and squashes the following commits:

509a233 [Cheng Hao] Remove the HiveQlQueryExecution
a5f9e3b [Cheng Hao] Remove the duplicated SparkSQLParser
@chenghao-intel chenghao-intel deleted the remove_sparksqlparser branch July 2, 2015 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@chenghao-intel @AmplabJenkins @SparkQA @marmbrus