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

Jacques Nadeau commented on DRILL-422:
--------------------------------------

Fixed in 0.6

> Use pseudo-injection buffer in function implementation when it requires 
> temporary buffer
> ----------------------------------------------------------------------------------------
>
>                 Key: DRILL-422
>                 URL: https://issues.apache.org/jira/browse/DRILL-422
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Jinfeng Ni
>             Fix For: 0.6.0
>
>
> In DRILL-379, the code uses a wrapper buffer over a java byte array for 
> temporary buffer used by some string functions. However, such wrapper buffer 
> will use big endian, different from Drill's native little endian. This may 
> not be a big issue for string, but could cause serious issues for other 
> types.  
> In stead, we should use an approach where the buffer is provided from Drill 
> fragment context through pseudo-injection to get the access to buffers, and 
> the context will be responsible for cleaning things up.



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

Reply via email to