[ https://issues.apache.org/jira/browse/CALCITE-2778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16737481#comment-16737481 ]
Julian Hyde commented on CALCITE-2778: -------------------------------------- Are you experiencing performance issues due to this class? (It's no longer used in Calcite, so I would be surprised.) Your fix removes the "set null" which, as the comment explained, is important to prevent cycles. > CompoundClosableAllocation Prefer ArrayList > ------------------------------------------- > > Key: CALCITE-2778 > URL: https://issues.apache.org/jira/browse/CALCITE-2778 > Project: Calcite > Issue Type: Improvement > Components: core > Reporter: BELUGA BEHR > Assignee: Julian Hyde > Priority: Minor > > In Java, ArrayList should be the default choice over LinkedList. It > generally requires less memory and is faster. Update > {{CompoundClosableAllocation}} to use an Array structure instead of linked > list and simplify the code. -- This message was sent by Atlassian JIRA (v7.6.3#76005)