[GitHub] spark pull request: [SPARK-11700] [SQL] use weak reference in Spar...

2015-11-26 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/9990#discussion_r46007594 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -401,7 +402,7 @@ class SQLContext private[sql]( */ @Experimental

[GitHub] spark pull request: [SPARK-11700] [SQL] use weak reference in Spar...

2015-11-26 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/9990#issuecomment-159967811 @zsxwing The one in SQLContext have a `clearActive` APIs, so I think don't need WeakRef there. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-11700] [SQL] use weak reference in Spar...

2015-11-26 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/9990#issuecomment-159982550 > @zsxwing The one in SQLContext have a clearActive APIs, so I think don't need WeakRef there. Sorry. I meant the compilation failed. --- If your project is