[jira] [Commented] (DRILL-8322) Add a list of scanned plugin names to the query profile

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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