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

    https://github.com/apache/spark/pull/19992#discussion_r157857840
  
    --- Diff: core/src/main/scala/org/apache/spark/storage/StorageLevel.scala 
---
    @@ -132,6 +132,10 @@ class StorageLevel private(
     
       override def hashCode(): Int = toInt * 41 + replication
     
    +  /** Name of the storage level if it is predefined or `None` otherwise. */
    +  def name: Option[String] = StorageLevel.PREDEFINED
    +    .collectFirst { case (storageLevel, name) if storageLevel == this => 
name }
    --- End diff --
    
    If the method body needs multiple lines it's better to wrap it with `{ }`. 
Same thing below.


---

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

Reply via email to