ychernysh commented on code in PR #2937:
URL: https://github.com/apache/drill/pull/2937#discussion_r1741159357
##########
exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/ParquetRecordReader.java:
##########
@@ -72,6 +73,11 @@ public class ParquetRecordReader extends
CommonParquetRecordReader {
private final boolean useBulkReader;
+ /**
+ * See {@link ParquetSchema#tableSchema}
+ */
+ private final TupleMetadata tableSchema;
Review Comment:
As I said before, this metadata is required for Drill to perform the work
assignment and we cannot avoid it. So, it is not optional and can't be turned
off.
This is the old parquet schema cache itself, if I understand correctly. Here
we pass it to parquet readers (it didn't actually provide a schema to the
readers, but we do now)
This is not related neither to Drill Metastore nor to provided schema
framework in any way.
--
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: dev-unsubscr...@drill.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
