Github user viirya commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19864#discussion_r156550811
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
    @@ -80,6 +80,14 @@ class CacheManager extends Logging {
         cachedData.isEmpty
       }
     
    +  private def extractStatsOfPlanForCache(plan: LogicalPlan): 
Option[Statistics] = {
    +    if (plan.stats.rowCount.isDefined) {
    --- End diff --
    
    I have the same question before. Seems @CodingCat's claim is that the 
`sizeInBytes` from relation can cause OOM issue: 
https://github.com/apache/spark/pull/19864#discussion_r155287758.
    



---

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

Reply via email to