Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14534#discussion_r73968962
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/server/SparkSQLOperationManager.scala
---
@@ -17,9 +17,8 @@
pa
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/14534#discussion_r73908742
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/server/SparkSQLOperationManager.scala
---
@@ -39,8 +39,10 @@ priva
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14534#discussion_r73834274
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/server/SparkSQLOperationManager.scala
---
@@ -39,8 +39,10 @@ private[th
GitHub user SaintBacchus opened a pull request:
https://github.com/apache/spark/pull/14534
[SPARK-16941]Add SynchronizedMap trait with Map in SparkSQLOperationManager.
## What changes were proposed in this pull request?
ThriftServer will have some thread-safe problem in
**SparkS
