This is an automated email from the ASF dual-hosted git repository.

desruisseaux pushed a commit to branch geoapi-4.0
in repository https://gitbox.apache.org/repos/asf/sis.git

commit 94f75c15288e03c89dc43fa8c4f447da390f4e11
Author: Martin Desruisseaux <martin.desruisse...@geomatys.com>
AuthorDate: Thu Sep 29 11:03:50 2022 +0200

    Add one more comment about the problem of using statistics for completing 
coverage metadata.
---
 .../main/java/org/apache/sis/internal/storage/StoreUtilities.java   | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/StoreUtilities.java
 
b/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/StoreUtilities.java
index eaeb355bab..495ff793e0 100644
--- 
a/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/StoreUtilities.java
+++ 
b/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/StoreUtilities.java
@@ -75,6 +75,12 @@ public final class StoreUtilities extends Static {
      * <p>This flag can be set to {@code true} for exploring data that we can 
not visualize otherwise.
      * But it should generally stay to {@code false}, because otherwise 
browsing resource metadata can
      * become as costly (slow and high memory usage) as visualizing the full 
raster.</p>
+     *
+     * <p>In addition of possible performance degradations, setting this flag 
to {@code true} can also prevent
+     * {@link org.apache.sis.storage.aggregate.CoverageAggregator} to group 
coverages that should be together.
+     * This is because using statistics may cause {@link 
org.apache.sis.coverage.SampleDimension} instances to
+     * have different sample value ranges for each coverage, which cause 
{@code CoverageAggregator} to consider
+     * that that can not be aggregated together.</p>
      */
     public static final boolean ALLOW_LAST_RESORT_STATISTICS = false;
 

Reply via email to