[ https://issues.apache.org/jira/browse/DRILL-340?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jason Altekruse updated DRILL-340: ---------------------------------- Component/s: Query Planning & Optimization > Ensure methods intended to create new copies of plan nodes are actually > creating copies of all nested objects > ------------------------------------------------------------------------------------------------------------- > > Key: DRILL-340 > URL: https://issues.apache.org/jira/browse/DRILL-340 > Project: Apache Drill > Issue Type: Bug > Components: Query Planning & Optimization > Reporter: Jason Altekruse > Fix For: Future > > > Methods such as getNewWithChildren declared in the PhysicalOperator interface > are designed to create copies of plan nodes. These copy operations should not > reference any part of the old nodes, thus child objects should be deep > copied, including lists of objects. For more information on the desired > functionality see here: http://en.wikipedia.org/wiki/Clone_(Java_method) > Existing implementations should be checked for correctness. We may want to > create this functionality for logical operators as well, but I do not believe > that we should need to do manipulations on logical plans that would require > it. Our optimization of plans will happen in the logical to physical plan > transformation. -- This message was sent by Atlassian JIRA (v6.3.4#6332)