cgivre commented on code in PR #2618:
URL: https://github.com/apache/drill/pull/2618#discussion_r945701382
##########
contrib/storage-splunk/src/main/java/org/apache/drill/exec/store/splunk/SplunkGroupScan.java:
##########
@@ -50,17 +54,20 @@ public class SplunkGroupScan extends AbstractGroupScan {
private int hashCode;
+ private MetadataProviderManager metadataProviderManager;
Review Comment:
This actually isn't related to the metastore. It is some boilerplate code
to allow for provided schema. You have to have the MetadataProviderManager to
access the inline schemata, which you then add to the schema negotiator in the
ScanBatchCreator class. This also is why I added all of the
`getPhysicalScan()` functions to the `SplunkStoragePlugin` class. With that
said, this PR does not go all the way and add provided schema support. I will
do that as a separate PR.
For future storage plugins, we should make that a standard, as it really
isn't difficult to add.
--
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]
