[
https://issues.apache.org/jira/browse/DRILL-8276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17579691#comment-17579691
]
ASF GitHub Bot commented on DRILL-8276:
---------------------------------------
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.
> Add Support for User Translation for Splunk
> -------------------------------------------
>
> Key: DRILL-8276
> URL: https://issues.apache.org/jira/browse/DRILL-8276
> Project: Apache Drill
> Issue Type: Task
> Components: Storage - Other
> Affects Versions: 1.20.2
> Reporter: Charles Givre
> Assignee: Charles Givre
> Priority: Major
> Fix For: 2.0.0
>
>
> This PR adds support for user translation to Splunk.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
