Skip to content

[SPARK-4769] [SQL] CTAS does not work when reading from temporary tables #3336

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

Closed
wants to merge 5 commits into from

Conversation

chenghao-intel
Copy link
Contributor

This is the code refactor and follow ups for #2570

@SparkQA
Copy link

SparkQA commented Nov 18, 2014

Test build #23536 has started for PR 3336 at commit 0e7770f.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 18, 2014

Test build #23536 has finished for PR 3336 at commit 0e7770f.

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

@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/23536/
Test FAILed.

@SparkQA
Copy link

SparkQA commented Nov 18, 2014

Test build #23549 has started for PR 3336 at commit 8e3b423.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 18, 2014

Test build #23549 has finished for PR 3336 at commit 8e3b423.

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

@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/23549/
Test FAILed.

@SparkQA
Copy link

SparkQA commented Nov 19, 2014

Test build #23575 has started for PR 3336 at commit 47f3a9d.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 19, 2014

Test build #23575 has finished for PR 3336 at commit 47f3a9d.

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

@AmplabJenkins
Copy link

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

@SparkQA
Copy link

SparkQA commented Dec 7, 2014

Test build #24214 has started for PR 3336 at commit b85b620.

  • This patch merges cleanly.

@chenghao-intel chenghao-intel changed the title [SPARK-4469] [SQL] Move the SemanticAnalyzer from Physical Execution to Analysis [SPARK-4769] [SQL] CTAS does not work when reading from temporary tables Dec 7, 2014
@SparkQA
Copy link

SparkQA commented Dec 7, 2014

Test build #24215 has started for PR 3336 at commit 5d58812.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 7, 2014

Test build #24216 has started for PR 3336 at commit 4f97f14.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 7, 2014

Test build #24215 has finished for PR 3336 at commit 5d58812.

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

@AmplabJenkins
Copy link

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

@SparkQA
Copy link

SparkQA commented Dec 7, 2014

Test build #24214 has finished for PR 3336 at commit b85b620.

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

@AmplabJenkins
Copy link

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

@SparkQA
Copy link

SparkQA commented Dec 7, 2014

Test build #24216 has finished for PR 3336 at commit 4f97f14.

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

@AmplabJenkins
Copy link

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

@chenghao-intel
Copy link
Contributor Author

@marmbrus Any comment on this?

@SparkQA
Copy link

SparkQA commented Dec 8, 2014

Test build #24217 has started for PR 3336 at commit e215187.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 8, 2014

Test build #24218 has started for PR 3336 at commit 3563142.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 8, 2014

Test build #24217 has finished for PR 3336 at commit e215187.

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

@AmplabJenkins
Copy link

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

@SparkQA
Copy link

SparkQA commented Dec 8, 2014

Test build #24218 has finished for PR 3336 at commit 3563142.

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

@AmplabJenkins
Copy link

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

asfgit pushed a commit that referenced this pull request Dec 9, 2014
This is the code refactor and follow ups for #2570

Author: Cheng Hao <hao.cheng@intel.com>

Closes #3336 from chenghao-intel/createtbl and squashes the following commits:

3563142 [Cheng Hao] remove the unused variable
e215187 [Cheng Hao] eliminate the compiling warning
4f97f14 [Cheng Hao] fix bug in unittest
5d58812 [Cheng Hao] revert the API changes
b85b620 [Cheng Hao] fix the regression of temp tabl not found in CTAS

(cherry picked from commit 51b1fe1)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in 51b1fe1 Dec 9, 2014
@chenghao-intel chenghao-intel deleted the createtbl branch July 2, 2015 08:41
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.

3 participants