[ 
https://issues.apache.org/jira/browse/SPARK-43523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17727189#comment-17727189
 ] 

Amine Bagdouri commented on SPARK-43523:
----------------------------------------

Hi [~srowen],

The purpose of the test is to illustrate the memory leak and be able to produce 
it quickly. It's not meant to be realistic. Regardles, this memory leak is 
something we have witnessed first hand on our production environment with a 
real life application : we have a driver with 8 gb of heap and it's memory 
became gradually full because of the leak within less than a week.

In my test, I have used a driver of only 1 gb of memory to quickly reach an 
OutOfMemory error. If I had used a driver with more memory (say 64 gb) I would 
have eventually arrived to the same result but it would have taken much more 
time.

Also, I'm indeed launching a lot of jobs. But, I'm launching them sequentially 
and not in parallel, to illustrate the point I'm trying to make here : every 
job execution leaves behind new objects that can't be garbage collected and 
that have no utility. I believe this is is by defintion a memory leak.

I'm quoting [this 
article|https://www.baeldung.com/java-memory-leaks#:~:text=A%20Memory%20Leak%20is%20a,degrades%20system%20performance%20over%20time]
 that provides a definition that matches exactly the issue I'm describing here 
: 
{code:java}
A Memory Leak is a situation where there are objects present in the heap that 
are no longer used, but the garbage collector is unable to remove them from 
memory, and therefore, they're unnecessarily maintained.
A memory leak is bad because it blocks memory resources and degrades system 
performance over time. If not dealt with, the application will eventually 
exhaust its resources, finally terminating with a fatal 
java.lang.OutOfMemoryError. {code}

> Memory leak in Spark UI
> -----------------------
>
>                 Key: SPARK-43523
>                 URL: https://issues.apache.org/jira/browse/SPARK-43523
>             Project: Spark
>          Issue Type: Bug
>          Components: Web UI
>    Affects Versions: 2.4.4, 3.4.0
>            Reporter: Amine Bagdouri
>            Priority: Major
>         Attachments: spark_shell_oom.log, spark_ui_memory_leak.zip
>
>
> We have a distributed Spark application running on Azure HDInsight using 
> Spark version 2.4.4.
> After a few days of active processing on our application, we have noticed 
> that the GC CPU time ratio of the driver is close to 100%. We suspected a 
> memory leak. Thus, we have produced a heap dump and analyzed it using Eclipse 
> Memory Analyzer.
> Here is some interesting data from the driver's heap dump (heap size is 8 GB):
>  * The estimated retained heap size of String objects (~5M instances) is 3.3 
> GB. It seems that most of these instances correspond to spark events.
>  * Spark UI's AppStatusListener instance estimated retained size is 1.1 GB.
>  * The number of LiveJob objects with status "RUNNING" is 18K, knowing that 
> there shouldn't be more than 16 live running jobs since we use a fixed size 
> thread pool of 16 threads to run spark queries.
>  * The number of LiveTask objects is 485K.
>  * The AsyncEventQueue instance associated to the AppStatusListener has a 
> value of 854 for dropped events count and a value of 10001 for total events 
> count, knowing that the dropped events counter is reset every minute and that 
> the queue's default capacity is 10000.
> We think that there is a memory leak in Spark UI. Here is our analysis of the 
> root cause of this leak:
>  * AppStatusListener is notified of Spark events using a bounded queue in 
> AsyncEventQueue.
>  * AppStatusListener updates its state (kvstore, liveTasks, liveStages, 
> liveJobs, ...) based on the received events. For example, onTaskStart adds a 
> task to liveTasks map and onTaskEnd removes the task from liveTasks map.
>  * When the rate of events is very high, the bounded queue in AsyncEventQueue 
> is full, some events are dropped and don't make it to AppStatusListener.
>  * Dropped events that signal the end of a processing unit prevent the state 
> of AppStatusListener from being cleaned. For example, a dropped onTaskEnd 
> event, will prevent the task from being removed from liveTasks map, and the 
> task will remain in the heap until the driver's JVM is stopped.
> We were able to confirm our analysis by reducing the capacity of the 
> AsyncEventQueue (spark.scheduler.listenerbus.eventqueue.capacity=10). After 
> having launched many spark queries using this config, we observed that the 
> number of active jobs in Spark UI increased rapidly and remained high even 
> though all submitted queries have completed. We have also noticed that some 
> executor task counters in Spark UI were negative, which confirms that 
> AppStatusListener state does not accurately reflect the reality and that it 
> can be a victim of event drops.
> Suggested fix:
> There are some limits today on the number of "dead" objects in 
> AppStatusListener's maps (for example: spark.ui.retainedJobs). We suggest 
> enforcing another configurable limit on the number of total objects in 
> AppStatusListener's maps and kvstore. This should limit the leak in the case 
> of high events rate, but AppStatusListener stats will remain inaccurate.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to