[ 
https://issues.apache.org/jira/browse/CALCITE-1012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15049593#comment-15049593
 ] 

Julian Hyde edited comment on CALCITE-1012 at 12/9/15 11:14 PM:
----------------------------------------------------------------

Makes sense.

Also, I think the JavaCC folks should consider converting from 1 array with 721 
JJCalls instances to 4 arrays (int[] gens, Token[] firsts, int[] args, int[] 
nexts). More friendly to both the JVM heap and the CPU's cache.


was (Author: julianhyde):
Makes sense.

Also, I think the JavaCC folks should consider converting from 1 array with 724 
JJCalls instances to 4 arrays. More friendly to the JVM heap and to the CPU's 
cache.

> Benchmark SQL parser
> --------------------
>
>                 Key: CALCITE-1012
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1012
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Julian Hyde
>            Assignee: Julian Hyde
>         Attachments: allocations1.png, allocations2.png, allocations3.png, 
> allocations4.png
>
>




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

Reply via email to