Actually I think the test in its existing form is already failing. I will
update it and send you an update.


On Wed, Jan 30, 2013 at 6:47 PM, James Taylor <notifications@github.com>wrote:

> In src/test/java/com/salesforce/phoenix/compile/QueryCompileTest.java:
>
> > +        List<Object> binds = Collections.emptyList();
> > +        Scan scan = new Scan();
> > +        compileQuery(query, binds, scan);
> > +        assertTrue(Bytes.compareTo(
> Bytes.toBytes("abc"), scan.getStartRow()) == 0);
> > +        assertTrue(Bytes.compareTo(Bytes.toBytes("abc\1"), scan.getStopRow()) == 0);
> > +        assertTrue(scan.getFilter() != null);
> > +    }
> > +
> > +    @Test
> > +    public void testRTrimSetScanKey() throws Exception {
> > +        String query = "SELECT inst FROM ptsdb WHERE rtrim(inst) = 'abc'";
> > +        List<Object> binds = Collections.emptyList();
> > +        Scan scan = new Scan();
> > +        compileQuery(query, binds, scan);
> > +        assertTrue(Bytes.compareTo(Bytes.toBytes("abc"), scan.getStartRow()) == 0);
> > +        assertTrue(Bytes.compareTo(Bytes.toBytes("abc\1"), scan.getStopRow()) == 0);
>
> Please update and then I'll pull it in. I'd rather not pull in tests
> that'll fail.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/forcedotcom/phoenix/pull/17/files#r2837937>.
>
>



-- 
Regards
-- Tony