Skip to content

Commit

Permalink
Review updates
Browse files Browse the repository at this point in the history
  • Loading branch information
aaime committed Feb 24, 2025
1 parent dd3248f commit fc0bcb0
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -105,10 +105,11 @@ public class CoverageViewReader implements GridCoverage2DReader {

/**
* Executor for parallel loading of coverages that do support multithreaded loading to start with (e.g., image
* msoaic). It's not limiting the number of threads, because when MT loading is one, that's already limited in the
* mosaic). It's not limiting the number of threads, because when MT loading is one, that's already limited in the
* readers. It cannot be the same executor used by the image mosaic (ResourcePool#getCoverageExecutor) because that
* would cause deadlock (each read would use at least 2 threads, one inside the image mosaic and one inside the
* coverage view). Should we switch to allow parlale
* coverage view). Should we switch to allow parellel loading of coverages that do not support MT natively, a
* separate executor should be used.
*/
private static final ExecutorService EXECUTOR = Executors.newCachedThreadPool();

Expand Down

0 comments on commit fc0bcb0

Please sign in to comment.