Github user amansinha100 commented on a diff in the pull request:
https://github.com/apache/drill/pull/531#discussion_r67914676
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/easy/EasyFormatPlugin.java
---
@@ -126,8 +127,12 @@ CloseableRecordBatch getReaderBatch(FragmentContext
context, EasySubScan scan) t
final ImplicitColumnExplorer columnExplorer = new
ImplicitColumnExplorer(context, scan.getColumns());
if (!columnExplorer.isSelectAllColumns()) {
+ // We must make sure to pass a table column (not to be confused with
implicit column) to the underlying record reader.
+ List<SchemaPath> tableColumns =
--- End diff --
I haven't looked at the original patch for implicit columns but I am not
sure why this fix is in the EasyFormatPlugin when the test is against Parquet
files ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
