[ 
https://issues.apache.org/jira/browse/SPARK-16928?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Davies Liu resolved SPARK-16928.
--------------------------------
       Resolution: Fixed
    Fix Version/s: 2.1.0

Issue resolved by pull request 14513
[https://github.com/apache/spark/pull/14513]

> Recursive call of ColumnVector::getInt() breaks JIT inlining
> ------------------------------------------------------------
>
>                 Key: SPARK-16928
>                 URL: https://issues.apache.org/jira/browse/SPARK-16928
>             Project: Spark
>          Issue Type: Improvement
>    Affects Versions: 2.0.0
>            Reporter: Qifan Pu
>            Priority: Minor
>             Fix For: 2.1.0
>
>
> In both OnHeapColumnVector and OffHeapColumnVector, we implemented getInt() 
> with the following code pattern: 
> {code}
>   public int getInt(int rowId) {
>     if (dictionary == null) {
>       return intData[rowId];
>     } else {
>       return dictionary.decodeToInt(dictionaryIds.getInt(rowId));
>     }
>   }
> {code}
> As dictionaryIds is also a ColumnVector, this results in a recursive call of 
> getInt() and breaks JIT inlining. As a result, getInt() will not get inlined.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to