CarbonDictionaryDecoder should support codegen

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

CarbonDictionaryDecoder should support codegen

bill.zhou
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.



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

Regards
Bill
Reply | Threaded
Open this post in threaded view
|

Re: CarbonDictionaryDecoder should support codegen

ravipesala
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