-
Notifications
You must be signed in to change notification settings - Fork 18
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
download specific dbs #19
Comments
No, this is not currently supported. Could you tell me a bit more about your usecase for this? |
Thanks for the super fast reply! |
Thanks for the explanation. Downloading only the country DB would save you about 37MB tops, which is really negligible on most systems those days, so I'm not entirely convinced it's worth adding a new feature for. Are you working in an IoT context? |
@GitSquared actually on my computer I'm seeing different values I don't feel this is negligible. I'm not using IOT, but I'm working on an app that should consume a lot of memory, and should be very fast, so I'm trying to save up as much as possible, and saving 77MB of memory on each container just by removing these DBs sounds great to me. Is there a reason not to implement this? I mean this feels like a basic feature request - download only what you need, no? |
Sorry, you're right — I was looking at the compressed archives that the lib download, not the size of the unpacked databases.
Note that if you're writing data to
It is pretty basic indeed, but I haven't touched this lib in a long time and wanted to clean it up a bit, so its a bit more of an investment on my end. But I guess i'll take the opportunity to finally do that ;) Should have something for you very soon, this week at least. |
That's amazing. thank you so much!
|
interesting, thanks for the link! |
Hey — just published https://github.com/GitSquared/node-geolite2-redist/releases/tag/v3.0.0 with this feat added! You can simply Lmk if you have any questions or trouble! |
Thanks so much for this awesome work! |
Hey, is there a way to only download a specific database and not all them?
The text was updated successfully, but these errors were encountered: