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

    https://github.com/apache/spark/pull/22814#discussion_r228381742
  
    --- Diff: docs/sql-data-sources-avro.md ---
    @@ -177,6 +180,19 @@ Data source options of Avro can be set using the 
`.option` method on `DataFrameR
       Currently supported codecs are <code>uncompressed</code>, 
<code>snappy</code>, <code>deflate</code>, <code>bzip2</code> and 
<code>xz</code>.<br> If the option is not set, the configuration 
<code>spark.sql.avro.compression.codec</code> config is taken into account.</td>
         <td>write</td>
       </tr>
    +  <tr>
    +    <td><code>mode</code></td>
    +    <td>FAILFAST</td>
    +    <td>The <code>mode</code> option allows to specify parse mode for 
function <code>from_avro</code>.<br>
    +      Currently supported modes are:
    +      <ul>
    +        <li><code>FAILFAST</code>: Throws an exception on processing 
corrupted record.</li>
    +        <li><code>PERMISSIVE</code>: Corrupt records are processed as null 
result. To implement this, the
    --- End diff --
    
    not a big deal but ..`To implement this` sounds a little bit awkward .. I 
think you can just say ... `Corrupt records are processed as null; therefore, 
the given schema is forced to be fully nullable.` (feel free to fix words as 
you think righter).


---

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

Reply via email to