spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vanzin <...@git.apache.org>
Subject [GitHub] spark pull request #20813: [SPARK-23670][SQL] Fix memory leak on SparkPlanGr...
Date Fri, 16 Mar 2018 00:03:33 GMT
Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20813#discussion_r174202008
  
    --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/ui/SQLAppStatusStore.scala
---
    @@ -53,6 +53,9 @@ class SQLAppStatusStore(
       def executionsCount(): Long = {
         store.count(classOf[SQLExecutionUIData])
       }
    +  def planGraphCount(): Long = {
    --- End diff --
    
    nit: add empty line before


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message