[
https://issues.apache.org/jira/browse/DRILL-4573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15263368#comment-15263368
]
jean-claude commented on DRILL-4573:
------------------------------------
Hey Sean,
You did not apply the correct patch. I made the code review changes you
asked and updated the patch in the jira ticket. But somehow the code that
made it into the git repo is the initial patch.
Have a look at the one in the jira ticket you will see it has comments in
the CharSequenceWrapper.
On Wed, Apr 27, 2016 at 10:12 PM, Sean Hsuan-Yi Chu (JIRA) <j...@apache.org>
> Zero copy LIKE, REGEXP_MATCHES, SUBSTR
> --------------------------------------
>
> Key: DRILL-4573
> URL: https://issues.apache.org/jira/browse/DRILL-4573
> Project: Apache Drill
> Issue Type: Improvement
> Reporter: jean-claude
> Priority: Minor
> Fix For: 1.7.0
>
> Attachments: DRILL-4573.patch.txt
>
>
> All the functions using the java.util.regex.Matcher are currently creating
> Java string objects to pass into the matcher.reset().
> However this creates unnecessary copy of the bytes and a Java string object.
> The matcher uses a CharSequence, so instead of making a copy we can create an
> adapter from the DrillBuffer to the CharSequence interface.
> Gains of 25% in execution speed are possible when going over VARCHAR of 36
> chars. The gain will be proportional to the size of the VARCHAR.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
