Agreed. I needed to fix the bug and get a release out with the minimum of disruption. There is an opportunity to do better as part of https://issues.apache.org/jira/browse/CALCITE-558. Please get involved in that.
Julian On Sun, Feb 1, 2015 at 5:00 AM, Vladimir Sitnikov <[email protected]> wrote: > +1. It compiles, and works for me. > > Julian, > > While your fix for 582 works, I do not like how it is implemented. > > If you suggest that *ableTable should always return Object[], then you > should express that in email/ticket or at least in > org.apache.calcite.adapter.enumerable.EnumerableTableScan#format. > > return ---> table.unwrap(QueryableTable.class) != null <--- && > fieldCount == 1 ? JavaRowFormat.SCALAR : JavaRowFormat.ARRAY; > > Vladimir
