-
Notifications
You must be signed in to change notification settings - Fork 11
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
GDALRasterSources performance #183
Comments
@notthatbreezy We talked about this yesterday in cross team planning. But we wanted to confirm the expected outcome, timeline, and priority for RF. Does it make sense to wait until https://github.com/azavea/raster-foundry-platform/issues/743 would be resolved, because this has the potential to fix the RF problems. It's an option to spend at least two points this sprint, but we confirm with you all. |
There are a few issues being worked on that probably need to be resolved for us to get more information to make working on GDAL related performance issues easier as far as RF goes.
I think at least the first two need to be done before we can start looking into garbage collection more otherwise we won't really have enough information to determine if we're improving things. |
RPMs for the EMR test are here: |
During the RasterSources benchmark https://github.com/pomadchin/vlm-performance/ on EMR we received no performance improvements with GDAL usage. On the contrary, it created more complexity in jobs configuration. Without tuning
GDAL_CAHEMAX
options andwo
(for warp operations), EMR nodes are dying / hanging and => tasks are failing. This is also can be the problem of the GDAL 2.3 usage onyarn
and according to OSGeo/gdal@5351335 GDAL doesn't respectcgroups
. But it is unclear howyarn
limits resources, it may be not usingcgroups
to limit CPU / RAM and this can be useless in this case.Related to https://github.com/azavea/raster-foundry-platform/issues/743
and to https://github.com/azavea/geotrellis/issues/68
The text was updated successfully, but these errors were encountered: