You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@drill.apache.org by "Jason Altekruse (JIRA)" <ji...@apache.org> on 2015/02/11 02:24:11 UTC

[jira] [Commented] (DRILL-2184) Cleanup TestFunctionsQuery

    [ https://issues.apache.org/jira/browse/DRILL-2184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14315347#comment-14315347 ] 

Jason Altekruse commented on DRILL-2184:
----------------------------------------

Even though the changes are simple, a patch this big should be put in reviewboard, I don't have a good way to refer to the code if I find anything wrong with it.

> Cleanup TestFunctionsQuery
> --------------------------
>
>                 Key: DRILL-2184
>                 URL: https://issues.apache.org/jira/browse/DRILL-2184
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Mehant Baid
>            Assignee: Mehant Baid
>            Priority: Minor
>         Attachments: DRILL-2184.patch
>
>
> Currently TestFunctionsQuery is present under the JDBC tests, it should be under exec and should use the new unit test framework for verification. Had to modify a couple of tests in this file as part of DRILL-2101 making this a good opportunity to clean this up.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)