GitHub user kiszk opened a pull request:

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

    [SPARK-22595][SQL] fix flaky test: CastSuite.SPARK-22500: cast for struct 
should not generate codes beyond 64KB

    ## What changes were proposed in this pull request?
    
    This PR reduces the number of fields in the test case of `CastSuite` to fix 
an issue that is pointed at 
[here](https://github.com/apache/spark/pull/19800#issuecomment-346634950).
    
    ```
    java.lang.OutOfMemoryError: GC overhead limit exceeded
    java.lang.OutOfMemoryError: GC overhead limit exceeded
        at org.codehaus.janino.UnitCompiler.findClass(UnitCompiler.java:10971)
        at 
org.codehaus.janino.UnitCompiler.findTypeByName(UnitCompiler.java:7607)
        at 
org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5758)
        at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5732)
        at org.codehaus.janino.UnitCompiler.access$13200(UnitCompiler.java:206)
        at 
org.codehaus.janino.UnitCompiler$18.visitReferenceType(UnitCompiler.java:5668)
        at 
org.codehaus.janino.UnitCompiler$18.visitReferenceType(UnitCompiler.java:5660)
        at org.codehaus.janino.Java$ReferenceType.accept(Java.java:3356)
        at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5660)
        at 
org.codehaus.janino.UnitCompiler.buildLocalVariableMap(UnitCompiler.java:2892)
        at org.codehaus.janino.UnitCompiler.compile(UnitCompiler.java:2764)
        at 
org.codehaus.janino.UnitCompiler.compileDeclaredMethods(UnitCompiler.java:1262)
        at 
org.codehaus.janino.UnitCompiler.compileDeclaredMethods(UnitCompiler.java:1234)
        at org.codehaus.janino.UnitCompiler.compile2(UnitCompiler.java:538)
        at org.codehaus.janino.UnitCompiler.compile2(UnitCompiler.java:890)
        at org.codehaus.janino.UnitCompiler.compile2(UnitCompiler.java:894)
        at org.codehaus.janino.UnitCompiler.access$600(UnitCompiler.java:206)
        at 
org.codehaus.janino.UnitCompiler$2.visitMemberClassDeclaration(UnitCompiler.java:377)
        at 
org.codehaus.janino.UnitCompiler$2.visitMemberClassDeclaration(UnitCompiler.java:369)
        at 
org.codehaus.janino.Java$MemberClassDeclaration.accept(Java.java:1128)
        at org.codehaus.janino.UnitCompiler.compile(UnitCompiler.java:369)
        at 
org.codehaus.janino.UnitCompiler.compileDeclaredMemberTypes(UnitCompiler.java:1209)
        at org.codehaus.janino.UnitCompiler.compile2(UnitCompiler.java:564)
        at org.codehaus.janino.UnitCompiler.compile2(UnitCompiler.java:890)
        at org.codehaus.janino.UnitCompiler.compile2(UnitCompiler.java:894)
        at org.codehaus.janino.UnitCompiler.access$600(UnitCompiler.java:206)
        at 
org.codehaus.janino.UnitCompiler$2.visitMemberClassDeclaration(UnitCompiler.java:377)
        at 
org.codehaus.janino.UnitCompiler$2.visitMemberClassDeclaration(UnitCompiler.java:369)
        at 
org.codehaus.janino.Java$MemberClassDeclaration.accept(Java.java:1128)
        at org.codehaus.janino.UnitCompiler.compile(UnitCompiler.java:369)
        at 
org.codehaus.janino.UnitCompiler.compileDeclaredMemberTypes(UnitCompiler.java:1209)
        at org.codehaus.janino.UnitCompiler.compile2(UnitCompiler.java:564)
    ...
    ```
    
    ## How was this patch tested?
    
    Used existing test case

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

    $ git pull https://github.com/kiszk/spark SPARK-22595

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

    https://github.com/apache/spark/pull/19806.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 #19806
    
----
commit 48103dd26817629cc00619ea8659d14533f53fb5
Author: Kazuaki Ishizaki <[email protected]>
Date:   2017-11-23T19:24:07Z

    reduce the number of fields to reduce GC pressure

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to