[GitHub] spark pull request #22989: [SPARK-25986][Build] Banning throw new OutOfMemor...

[felixcheung]

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

    https://github.com/apache/spark/pull/22989#discussion_r232477783
  
    --- Diff: scalastyle-config.xml ---
    @@ -240,6 +240,18 @@ This file is divided into 3 sections:
         ]]></customMessage>
       </check>
     
    +  <check customId="notthrowoutofmemory" level="error" 
class="org.scalastyle.file.RegexChecker" enabled="true">
    +    <parameters><parameter name="regex">throw new 
OutOfMemoryError</parameter></parameters>
    +    <customMessage><![CDATA[
    +      Are you sure that you want to throw OutOfMemoryError? It will kill 
the hole executor, you should use
    --- End diff --
    
    perhasp "it will kill the whole executor" is a bit hard to understand? (I 
mean, is there a way to kill part of the executor?)


---

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