HyukjinKwon commented on a change in pull request #29331:
URL: https://github.com/apache/spark/pull/29331#discussion_r464220055
##########
File path: core/src/main/scala/org/apache/spark/storage/StorageLevel.scala
##########
@@ -172,6 +173,7 @@ object StorageLevel {
case "NONE" => NONE
case "DISK_ONLY" => DISK_ONLY
case "DISK_ONLY_2" => DISK_ONLY_2
+ case "DISK_ONLY_3" => DISK_ONLY_3
Review comment:
Maybe we should fix Java, Python and R sides and the documentation
together given that the changes are not so big. I checked it in
https://github.com/apache/spark/search?q=DISK_ONLY_2&unscoped_q=DISK_ONLY_2.
BTW, is it worth adding this? It was already possible before via using
user-defined `StorageLevel`, and adding this seems making this list
inconsistent.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
