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-8710] [SQL] Change ScalaReflection.mirror from a val to a def. #7094
Conversation
Merged build triggered. |
Merged build started. |
Merged build finished. Test FAILed. |
@@ -28,7 +28,8 @@ import org.apache.spark.sql.types._ | |||
*/ | |||
object ScalaReflection extends ScalaReflection { | |||
val universe: scala.reflect.runtime.universe.type = scala.reflect.runtime.universe | |||
val mirror: universe.Mirror = universe.runtimeMirror(Thread.currentThread().getContextClassLoader) | |||
override def mirror: universe.Mirror = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add inline comment explaining why this needs to be a def?
Merged build triggered. |
Merged build started. |
Test build #36037 has started for PR 7094 at commit |
lgtm |
Test build #36037 has finished for PR 7094 at commit
|
Merged build finished. Test PASSed. |
Merged. |
jira: https://issues.apache.org/jira/browse/SPARK-8710 Author: Yin Huai <yhuai@databricks.com> Closes #7094 from yhuai/SPARK-8710 and squashes the following commits: c854baa [Yin Huai] Change ScalaReflection.mirror from a val to a def. (cherry picked from commit 4b497a7) Signed-off-by: Reynold Xin <rxin@databricks.com>
jira: https://issues.apache.org/jira/browse/SPARK-8710