[GROOVY-6523] Improve Documentation for in DefaultGroovyMethods.intersect(..)

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.2.0
• Fix Version/s: 2.5.0-rc-2
• Component/s: Documentation
• Assignee: Paul King
• Reporter: Alex Beardsley
• Created: 16/Jan/14 10:44
• Updated: 26/Jul/18 04:38
• Resolved: 16/Apr/18 05:29

Description

As noted in GROOVY-1986, the current version of this method is slow compared to Java's existing retainAll method for Sets, on the order of 4000% in test cases with a significant amount of data and intersections. The performance can't be improved due to limitations in the Java API.

This should be documented, as we would have never chosen intersect(..) if we were aware of the performance implications vs. convenience it provides.