CarbonDictionaryDecoder should support codegen

["bill.zhou" <zg...@163.com>]

hi All
   Now for the canrbon scan support codegen, but carbonditionarydecoder
does't support codegen, I think it should support. 
   For example, toady I do one test and the query plan is as following left,
if CarbondictionaryDecoder support codegen the plan will change to following
right.  I think that will improve the performance. 

<http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/file/n8600/3.png> 

Please Ravindra,Jacky,Liang correct it.  thank you. 

Regards
Bill 



--
View this message in context: http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/CarbonDictionaryDecoder-should-support-codegen-tp8600.html
Sent from the Apache CarbonData Mailing List archive mailing list archive at Nabble.com.

Re: CarbonDictionaryDecoder should support codegen

[Ravindra Pesala <ra...@gmail.com>]

Thanks Bill for pointing out.
Yes it is a long pending task and we should do it. I am not not sure about
the performance benefit we get, but definitely we should try. I will try it
out and see.

Regards,
Ravindra.

On 10 March 2017 at 17:18, bill.zhou <zg...@163.com> wrote:

> hi All
>    Now for the canrbon scan support codegen, but carbonditionarydecoder
> does't support codegen, I think it should support.
>    For example, toady I do one test and the query plan is as following
> left,
> if CarbondictionaryDecoder support codegen the plan will change to
> following
> right.  I think that will improve the performance.
>
> <http://apache-carbondata-mailing-list-archive.1130556.
> n5.nabble.com/file/n8600/3.png>
>
> Please Ravindra,Jacky,Liang correct it.  thank you.
>
> Regards
> Bill
>
>
>
> --
> View this message in context: http://apache-carbondata-
> mailing-list-archive.1130556.n5.nabble.com/CarbonDictionaryDecoder-
> should-support-codegen-tp8600.html
> Sent from the Apache CarbonData Mailing List archive mailing list archive
> at Nabble.com.
>



-- 
Thanks & Regards,
Ravi