[jira] [Commented] (DRILL-2269) Provide default implementation for getting cost of evaluating an expression in base class of LogicalExpression, in stead of throw Exception

2015-02-18 Thread Aman Sinha (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-2269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14326797#comment-14326797 ] Aman Sinha commented on DRILL-2269: --- Test output looks good. +1. > Provide default im

[jira] [Commented] (DRILL-2269) Provide default implementation for getting cost of evaluating an expression in base class of LogicalExpression, in stead of throw Exception

2015-02-18 Thread Jinfeng Ni (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-2269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14326792#comment-14326792 ] Jinfeng Ni commented on DRILL-2269: --- Test with hive function "ucase" and "lcase" in a fi

[jira] [Commented] (DRILL-2269) Provide default implementation for getting cost of evaluating an expression in base class of LogicalExpression, in stead of throw Exception

2015-02-18 Thread Aman Sinha (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-2269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14326739#comment-14326739 ] Aman Sinha commented on DRILL-2269: --- +1 on the patch. Could you a manual verification u