[
https://issues.apache.org/jira/browse/DRILL-8322?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17613198#comment-17613198
]
ASF GitHub Bot commented on DRILL-8322:
---------------------------------------
jnturton commented on code in PR #2661:
URL: https://github.com/apache/drill/pull/2661#discussion_r985959030
##########
exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java:
##########
@@ -87,6 +89,7 @@ public enum SqlStatementType {OTHER, ANALYZE, CTAS, EXPLAIN,
DESCRIBE_TABLE, DES
/** Stores constants and their holders by type */
private final Map<String, Map<MinorType, ValueHolder>>
constantValueHolderCache;
private SqlStatementType stmtType;
+ private Collection<StoragePlugin> scannedPlugins;
Review Comment:
I wasn't sure whether this new member is appropriate here.
> Add a list of scanned plugin names to the query profile
> -------------------------------------------------------
>
> Key: DRILL-8322
> URL: https://issues.apache.org/jira/browse/DRILL-8322
> Project: Apache Drill
> Issue Type: Improvement
> Components: Server
> Affects Versions: 1.20.2
> Reporter: James Turton
> Priority: Minor
> Fix For: 2.0.0
>
>
> A useful piece of information about a query is the set of plugins that it
> attempted to scan. While some scan operator information is present in the
> physical plan text in the query profile, an easy to parse, sorted and
> deduplicated list of plugin names is not.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
