arina-ielchiieva commented on a change in pull request #1819: DRILL-7313: Use
Hive schema for MaprDB native reader when field was empty
URL: https://github.com/apache/drill/pull/1819#discussion_r299884363
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/metastore/FileSystemMetadataProviderManager.java
##########
@@ -38,6 +41,34 @@ public static MetadataProviderManager init() {
return new FileSystemMetadataProviderManager();
}
+ /**
+ * Returns {@link TableMetadataProvider} which provides specified schema.
+ *
+ * @param schema table schema which should be provided
+ * @return {@link TableMetadataProvider} which provides specified schema
+ * @throws IOException if exception during {@link TableMetadataProvider}
construction happened
+ */
+ public static TableMetadataProvider
getMetadataProviderForSchema(TupleMetadata schema) throws IOException {
+ return new
FileSystemMetadataProviderManager().builder(MetadataProviderKind.SCHEMA_STATS_ONLY)
+ .withSchema(schema)
+ .build();
+ }
+
+ /**
+ * Checks whether specified {@link MetadataProviderManager} is not null and
returns {@link TableMetadataProvider}
+ * obtained from specified {@link MetadataProviderManager}.
+ * Otherwise {@link FileSystemMetadataProviderManager} is used for
constructing {@link TableMetadataProvider}.
Review comment:
```suggestion
* Otherwise {@link FileSystemMetadataProviderManager} is used to
construct {@link TableMetadataProvider}.
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
