Skip to content

Conversation

ravipesala
Copy link
Contributor

The below query gives error
sql("SELECT k FROM (SELECT key AS k FROM src) a")
It gives error because the aliases are not cleaned so it could not be resolved in further processing.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@marmbrus
Copy link
Contributor

marmbrus commented Oct 1, 2014

ok to test

@AmplabJenkins
Copy link

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

@ravipesala
Copy link
Contributor Author

@marmbrus I am not sure why it is failed. The error shows git could not fetch the code and timed out. Do I have to do something here.?

@marmbrus
Copy link
Contributor

marmbrus commented Oct 1, 2014

Git hub can be surprisingly flakey. Feel free to ping me when this happens and I'll restart the tests.

@SparkQA
Copy link

SparkQA commented Oct 1, 2014

QA tests have started for PR 2594 at commit d55db54.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 1, 2014

QA tests have finished for PR 2594 at commit d55db54.

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

@marmbrus
Copy link
Contributor

marmbrus commented Oct 1, 2014

Nice clean fix :)

Thanks for doing this! Merged to master.

@asfgit asfgit closed this in 3508ce8 Oct 1, 2014
marmbrus pushed a commit to marmbrus/spark that referenced this pull request Nov 11, 2014
The below query gives error
sql("SELECT k FROM (SELECT \`key\` AS \`k\` FROM src) a")
It gives error because the aliases are not cleaned so it could not be resolved in further processing.

Author: ravipesala <[email protected]>

Closes apache#2594 from ravipesala/SPARK-3708 and squashes the following commits:

d55db54 [ravipesala] Fixed SPARK-3708 (Backticks aren't handled correctly is aliases)

Conflicts:
	sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
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