Skip to content

Conversation

@yhuai
Copy link
Contributor

@yhuai yhuai commented Sep 20, 2016

@yhuai
Copy link
Contributor Author

yhuai commented Sep 20, 2016

cc @vanzin

@SparkQA
Copy link

SparkQA commented Sep 20, 2016

Test build #65624 has finished for PR 15157 at commit 5b73205.

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

@vanzin
Copy link
Contributor

vanzin commented Sep 20, 2016

If it's not too much trouble, keeping the workaround in CodeGenerator.scala would be nice. Otherwise LGTM.

@yhuai
Copy link
Contributor Author

yhuai commented Sep 20, 2016

Oh, right. Will do that.

On Tue, Sep 20, 2016 at 8:57 AM -0700, "Marcelo Vanzin" [email protected] wrote:

If it's not too much trouble, keeping the workaround in CodeGenerator.scala would be nice. Otherwise LGTM.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

… cached relation."

This reverts changes made to InMemoryRelation and InMemoryColumnarQuerySuite in commit 39e2bad.
But, it keeps the change in CodeGenerator.scala to make recordCompilationStats tolerant to errors thrown by janino.
@yhuai
Copy link
Contributor Author

yhuai commented Sep 20, 2016

Done.

@yhuai
Copy link
Contributor Author

yhuai commented Sep 20, 2016

I will merge this PR to master and branch 2.0 once it passes jenkins.

@SparkQA
Copy link

SparkQA commented Sep 20, 2016

Test build #65669 has finished for PR 15157 at commit 7df5d58.

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

@yhuai
Copy link
Contributor Author

yhuai commented Sep 20, 2016

@vanzin I am merging this PR to master and branch 2.0

asfgit pushed a commit that referenced this pull request Sep 20, 2016
@asfgit asfgit closed this in 9ac68db Sep 20, 2016
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