GitHub user kiszk opened a pull request:

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

    [SPARK-23032][SQL][FOLLOW-UP]Add codegenStageId in comment

    ## What changes were proposed in this pull request?
    
    This PR always adds `codegenStageId` in comment of the generated class.
    
    ```
    /* 001 */ public Object generate(Object[] references) {
    /* 002 */   return new GeneratedIteratorForCodegenStage1(references);
    /* 003 */ }
    /* 004 */
    /* 005 */ final class GeneratedIteratorForCodegenStage1 extends 
org.apache.spark.sql.execution.BufferedRowIterator {
    /* 006 */   // id=1
    /* 007 */   private Object[] references;
    ...
    ```
    
    ## How was this patch tested?
    
    Existing tests

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

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

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

    https://github.com/apache/spark/pull/20419.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 #20419
    
----
commit 6ee910611e7b6311f4ec1c5afb0ce5274fcf6805
Author: Kazuaki Ishizaki <ishizaki@...>
Date:   2018-01-29T00:35:06Z

    initial commit

----


---

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

Reply via email to