Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use fastutil-core or include only needed fastutil classes #2

Open
Marcono1234 opened this issue Sep 13, 2022 · 0 comments
Open

Use fastutil-core or include only needed fastutil classes #2

Marcono1234 opened this issue Sep 13, 2022 · 0 comments
Labels
performance-investigation Issue / pull request which is about investigating improving the performance.

Comments

@Marcono1234
Copy link
Owner

The fastutil dependency is currently ~20 MB large, however this project only uses a small fraction of the available classes. It might therefore be worth investigating whether it is possible to reduce this memory usage:

  • Using fastutil-core
    fastutil-core only includes a fraction of all classes available in the regular fastutil artifact. However, it has these problems:
    • It might not contain the classes needed by this project
    • Because it contains the same classes as the regular fastutil artifact, it could lead to issues for consuming projects when they have both dependencies on their classpath (and tiny-lingua cannot do much / anything to prevent this).
  • Shading the needed classes from fastutil into tiny-lingua
    • Have to check whether the fastutil license permits this, and whether that makes licensing / distributing tiny-lingua more difficult
    • Maintaining the list of classes to include / exclude will be error-prone
@Marcono1234 Marcono1234 added the performance-investigation Issue / pull request which is about investigating improving the performance. label Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance-investigation Issue / pull request which is about investigating improving the performance.
Projects
None yet
Development

No branches or pull requests

1 participant