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 <zgcsk...@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
