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 <
[hidden email]> 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