Hi Tim, Could post some sample code as to how to test a SQL query that returns a Boolean? —C
> On Jun 28, 2018, at 17:30, Timothy Farkas <tfar...@mapr.com> wrote: > > - We would have to add a boolean column reader to ColumnAccessors and wire > it in and add a getBoolean method to ScalarReader. > > - Your example should work as is, ClusterTest has a testBuilder method > that allows you to use the traditional test builder. Is there something not > working with the test builder? > > Tim > > > On Thu, Jun 28, 2018 at 12:39 PM, Arina Yelchiyeva < > arina.yelchiy...@gmail.com> wrote: > >> Hi Tim, >> >> it looks like deprecating BaseTestQuery was a little bit pre-mature. >> For example, from in this PR - https://urldefense.proofpoint. >> com/v2/url?u=https-3A__github.com_apache_drill_pull_1331&d=DwIBaQ&c= >> cskdkSMqhcnjZxdQVpwTXg&r=4eQVr8zB8ZBff-yxTimdOQ&m= >> zoNJPdWKxMX9-jbR2bftzwkX-CSrihIbpCirhHM1kA0&s=_uxhA_ >> qiMBTjbit6DDw-DvZNRGesfeA5g-FQjkl7f10&e= - >> Charles is trying to re-work BaseTestQuery usage to ClusterTest. >> First, it did not contain getSigletonDouble method which Charles has >> implemented. Now he has troubles with implementing getSigletonBoolean >> method which might be due to reader limitations. >> Also I am not quite clear how we can verify columns names and multiple >> columns in the result. >> For example: >> >> testBuilder() >> .sqlQuery("select (mi || lname) as CONCATOperator, mi, lname, >> concat(mi, lname) as CONCAT from concatNull") >> .ordered() >> .baselineColumns("CONCATOperator", "mi", "lname", "CONCAT") >> .baselineValues("A.Nowmer", "A.", "Nowmer", "A.Nowmer") >> .baselineValues("I.Whelply", "I.", "Whelply", "I.Whelply") >> .baselineValues(null, null, "Derry", "Derry") >> .baselineValues("J.Spence", "J.", "Spence", "J.Spence") >> .build().run(); >> >> Can you please suggest how this example can be re-written? >> >> Kind regards, >> Arina >> >> On Mon, Jun 25, 2018 at 11:10 PM Timothy Farkas <tfar...@mapr.com> wrote: >> >>> Hi All, >>> >>> BaseTestQuery was deprecated a while ago. Keeping it short and sweet :), >> if >>> you want to use BaseTestQuery directly, don't. Use ClusterTest instead. >> If >>> you are using PlanTestBase for planner tests, continue to do so. >> Eventually >>> PlanTestBase will be changed to extend ClusterTest instead. There is a >> JIRA >>> to track that issue https://urldefense.proofpoint. >> com/v2/url?u=https-3A__issues.apache.org_jira_browse_DRILL- >> 2D6536&d=DwIBaQ&c=cskdkSMqhcnjZxdQVpwTXg&r=4eQVr8zB8ZBff-yxTimdOQ&m= >> zoNJPdWKxMX9-jbR2bftzwkX-CSrihIbpCirhHM1kA0&s= >> BPAlenAq0k1kjAz7fUYXyEQMaJM1IPOgmdeySMMY84U&e=. >>> >>> Thanks, >>> Tim >>> >>