dzamo opened a new pull request #2267:
URL: https://github.com/apache/drill/pull/2267
This commit adds support for java.lang.Float to PojoWriters.java and a
test for the problem described in DRILL-7968 to TestMetastoreCommands.java.
For tables with fewer than ~200 rows and a FLOAT4 column there is no bug:
the ANALYZE command succeeds and, indeed, this case is exercised by tests in
TestMetastoreCommands.java and alltypes_{required,optional}.parquet which both
contain a FLOAT4 column.
But for tables with more than ~200 rows and a FLOAT4 column the ANALYZE
command fails with
```
SQL Error: EXECUTION_ERROR ERROR: PojoRecordReader doesn't yet support
conversions from the type [class java.lang.Float].
Failed to setup reader: DynamicPojoRecordReader
```
E.g. you can reproduce the above with
```sql
create table dfs.tmp.test_analyze as
select cast(1 as float) from cp.`employee.json`;
analyze table dfs.tmp.test_analyze refresh metadata;
```
## Documentation
No user-visible change.
## Testing
TestMetastoreCommands.java and ad hoc SQL testing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
