[ 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)