Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/531#discussion_r67916478
--- 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 --
In original PR I have created helper class which contained common logic
for parquet and test format plugins. Somehow I missed that this part is unique
for text format plugin, and should NOT be used in parquet one. That's why I
have removed it from ImplicitColumnExplorer and added to EasyFormatPlugin.
---
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.
---
