GitHub user dilipbiswal opened a pull request:

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

    [SPARK-25307] ArraySort function may return an error in the code generation 
phase

    ## What changes were proposed in this pull request?
    Sorting array of booleans (not nullable) returns a compilation error in the 
code generation phase. Below is the compilation error :
    ```SQL
    java.util.concurrent.ExecutionException: 
org.codehaus.commons.compiler.CompileException: File 'generated.java', Line 51, 
Column 23: failed to compile: org.codehaus.commons.compiler.CompileException: 
File 'generated.java', Line 51, Column 23: No applicable constructor/method 
found for actual parameters "boolean[]"; candidates are: "public static void 
java.util.Arrays.sort(long[])", "public static void 
java.util.Arrays.sort(long[], int, int)", "public static void 
java.util.Arrays.sort(byte[], int, int)", "public static void 
java.util.Arrays.sort(float[])", "public static void 
java.util.Arrays.sort(float[], int, int)", "public static void 
java.util.Arrays.sort(char[])", "public static void 
java.util.Arrays.sort(char[], int, int)", "public static void 
java.util.Arrays.sort(short[], int, int)", "public static void 
java.util.Arrays.sort(short[])", "public static void 
java.util.Arrays.sort(byte[])", "public static void 
java.util.Arrays.sort(java.lang.Object[], int, int, java.util.Compar
 ator)", "public static void java.util.Arrays.sort(java.lang.Object[], 
java.util.Comparator)", "public static void java.util.Arrays.sort(int[])", 
"public static void java.util.Arrays.sort(java.lang.Object[], int, int)", 
"public static void java.util.Arrays.sort(java.lang.Object[])", "public static 
void java.util.Arrays.sort(double[])", "public static void 
java.util.Arrays.sort(double[], int, int)", "public static void 
java.util.Arrays.sort(int[], int, int)"
        at 
com.google.common.util.concurrent.AbstractFuture$Sync.getValue(AbstractFuture.java:306)
        at 
com.google.common.util.concurrent.AbstractFuture$Sync.get(AbstractFuture.java:293)
        at 
com.google.common.util.concurrent.AbstractFuture.get(AbstractFuture.java:116)
        at 
com.google.common.util.concurrent.Uninterruptibles.getUninterruptibly(Uninterruptibles.java:135)
        at 
com.google.common.cache.LocalCache$Segment.getAndRecordStats(LocalCache.java:2410)
        at 
com.google.common.cache.LocalCache$Segment.loadSync(LocalCache.java:2380)
        at 
com.google.common.cache.LocalCache$Segment.lockedGetOrLoad(LocalCache.java:2342)
        at com.google.common.cache.LocalCache$Segment.get(LocalCache.java:2257)
        at com.google.common.cache.LocalCache.get(LocalCache.java:4000)
        at com.google.common.cache.LocalCache.getOrLoad(LocalCache.java:4004)
        at 
com.google.common.cache.LocalCache$LocalLoadingCache.get(LocalCache.java:4874)
        at 
org.apache.spark.sql.catalyst.expressions.codegen.CodeGenerator$.compile(CodeGenerator.scala:1305)
    
    ```
    
    ## How was this patch tested?
    Added test in collectionExpressionSuite

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

    $ git pull https://github.com/dilipbiswal/spark SPARK-25307

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

    https://github.com/apache/spark/pull/22314.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 #22314
    
----
commit 7dec58174e9901568253e5cd3ade528b30529317
Author: Dilip Biswal <dbiswal@...>
Date:   2018-08-31T03:17:39Z

    [SPARK-25307] ArraySort function may return an error in the code generation 
phase.

----


---

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

Reply via email to