ytatichno commented on PR #652: URL: https://github.com/apache/commons-collections/pull/652#issuecomment-4730204711
This problem (controlling the initial capacity of the inverted map) could be elegantly dodged if we had an overload that accepts a target map, similar to the `MultiMapUtils.invert` method proposed in [next PR [COLLECTIONS-880]](https://github.com/apache/commons-collections/pull/668). The caller could then pre-size the map as they wish, and the library wouldn’t need to guess the right capacity. Not a blocker for this PR - the current patch already fixes the immediate performance issue - but maybe worth considering as a follow-up. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
