GitHub user 10110346 opened a pull request:

    https://github.com/apache/spark/pull/22754

    [CORE][MINOR]The disk write buffer size must be greater than 12

    ## What changes were proposed in this pull request?
    
     In `UnsafeSorterSpillWriter.java`, when we write a record to a spill file 
wtih ` void write(Object baseObject, long baseOffset,  int recordLength, long 
keyPrefix)`, `recordLength` and `keyPrefix`  will be  written  the disk write 
buffer  first, and these will take 12 bytes, so the disk write buffer size must 
be greater than 12.
    
    ## How was this patch tested?
    Existing UT in `UnsafeExternalSorterSuite`


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/10110346/spark diskWriteBufferSize

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/22754.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #22754
    
----
commit dfc4b65088602ee45e0babe22e64e205fab3e82b
Author: liuxian <liu.xian3@...>
Date:   2018-10-17T06:03:00Z

    fix

----


---

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

Reply via email to