Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Drill, mail # dev - Review Request 17881: Drill 363


Copy link to this message
-
Re: Review Request 17881: Drill 363
Aman Sinha 2014-02-18, 04:21

This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17881/#review34680

Ship it!
Changes look good to me.  One clarification: your comments in the description mention changes to the DefaultFunctionResolver but I did not see that in the diffs.  Did you mean the cost adjustment done in the TypeCastRules ? The code change there looks reasonable to me.  

- Aman Sinha
On Feb. 9, 2014, 11:48 p.m., Jason Altekruse wrote: