[ https://issues.apache.org/jira/browse/DRILL-6476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16507241#comment-16507241 ]
ASF GitHub Bot commented on DRILL-6476: --------------------------------------- amansinha100 commented on a change in pull request #1308: DRILL-6476: Generate explain plan which shows relation between Latera… URL: https://github.com/apache/drill/pull/1308#discussion_r194244008 ########## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/explain/NumberingRelWriter.java ########## @@ -47,6 +50,7 @@ private final SqlExplainLevel detailLevel; protected final Spacer spacer = new Spacer(); private final List<Pair<String, Object>> values = new ArrayList<>(); + private final Map<String, Prel> registry; Review comment: The term 'registry' seems somewhat generic since this functionality is only used for the special case of Correlate-Unnest. Perhaps sourceOperatorRegistry or simply sourceOperatorMap ? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Generate explain plan which shows relation between Lateral and the > corresponding Unnest. > ---------------------------------------------------------------------------------------- > > Key: DRILL-6476 > URL: https://issues.apache.org/jira/browse/DRILL-6476 > Project: Apache Drill > Issue Type: Bug > Components: Query Planning & Optimization > Affects Versions: 1.14.0 > Reporter: Hanumath Rao Maduri > Assignee: Hanumath Rao Maduri > Priority: Major > > Currently, explain plan doesn't show that which lateral and unnest node's > are related. This information is good to have so that the visual plan can use > it and show the relation visually. > > -- This message was sent by Atlassian JIRA (v7.6.3#76005)