donraab commented on issue #40896: URL: https://github.com/apache/arrow/issues/40896#issuecomment-2049114181
@lidavidm You're welcome. I was curious to understand why Apache Arrow had a dependency on Eclipse Collections which is how I found this issue, albeit a bit too late unfortunately to save the effort. Using one of the primitive collections like `IntObjectHashMap` is a fairly common reason some folks first introduce Eclipse Collections as a dependency. I certainly appreciate every project's desire and challenge to reduce runtime dependencies. I found the PR that introduced the Eclipse Collections dependency six months ago and it simultaneously removed Netty Common as a dependency. There was a specific difference in the `values()` implementation in `MapWithOrdinalImpl` when the `IntObjectHashMap` from Netty was previously used. I don't know why the different implementation using `StreamSupport` with Apache Arrow `Preconditions` was needed to build the `values` collection. It might be worth contacting the developer who originally swapped the Netty `IntObjectMap` out for the Eclipse Collections `IntObjectMap` now that the code base includes a copy of the Netty IntObjectMap. @martin-traverse I would appreciate if you would let me know if there is an issue with Gradle License Report picking up dual licenses, or if there is something we can address in Eclipse Collections to make sure this doesn't happen to others in the future thinking they have to use the EPL license when the EDL license is there to provide flexibility to consumers of the library. Thanks! -- 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]
