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
The following commit(s) were added to refs/heads/geoapi-4.0 by this push:
new 3f35250 Minor tuning: - Remove unnecessary synchronization. - Forward
loading strategy to wrapped resources. - Typos.
3f35250 is described below
commit 3f352507cf5664f71bb6677eac22d6c9539630b7
Author: Martin Desruisseaux <martin.desruisse...@geomatys.com>
AuthorDate: Sun Oct 24 12:12:38 2021 +0200
Minor tuning:
- Remove unnecessary synchronization.
- Forward loading strategy to wrapped resources.
- Typos.
---
.../org/apache/sis/filter/DefaultFilterFactory.java | 6 +++---
.../java/org/apache/sis/storage/CoverageSubset.java | 20 +++++++++++++++++++-
2 files changed, 22 insertions(+), 4 deletions(-)
diff --git
a/core/sis-feature/src/main/java/org/apache/sis/filter/DefaultFilterFactory.java
b/core/sis-feature/src/main/java/org/apache/sis/filter/DefaultFilterFactory.java
index b228bd8..b9b495a 100644
---
a/core/sis-feature/src/main/java/org/apache/sis/filter/DefaultFilterFactory.java
+++
b/core/sis-feature/src/main/java/org/apache/sis/filter/DefaultFilterFactory.java
@@ -80,7 +80,7 @@ public abstract class DefaultFilterFactory<R,G,T> extends
AbstractFactory implem
private final Map<String,FunctionRegister> availableFunctions;
/**
- * Creates a new factory for geometries of temporal objects of the given
types.
+ * Creates a new factory for geometries and temporal objects of the given
types.
* The {@code spatial} argument can be one of the following classes:
*
* <table class="sis">
@@ -130,7 +130,7 @@ public abstract class DefaultFilterFactory<R,G,T> extends
AbstractFactory implem
*
* @todo The type of temporal object is not yet determined.
*/
- public static synchronized FilterFactory<Feature, Object, Object>
forFeatures() {
+ public static FilterFactory<Feature, Object, Object> forFeatures() {
return Features.DEFAULT;
}
@@ -196,7 +196,7 @@ public abstract class DefaultFilterFactory<R,G,T> extends
AbstractFactory implem
* of a resource between the specified dates.
*
* @param identifier identifier of the resource that shall be
selected by the predicate.
- * @param version version of the resource shall be selected, or
{@code null} for any version.
+ * @param version version of the resource to select, or {@code
null} for any version.
* @param startTime start time of the resource to select, or {@code
null} if none.
* @param endTime end time of the resource to select, or {@code
null} if none.
* @return the predicate.
diff --git
a/storage/sis-storage/src/main/java/org/apache/sis/storage/CoverageSubset.java
b/storage/sis-storage/src/main/java/org/apache/sis/storage/CoverageSubset.java
index b93ef03..47333b6 100644
---
a/storage/sis-storage/src/main/java/org/apache/sis/storage/CoverageSubset.java
+++
b/storage/sis-storage/src/main/java/org/apache/sis/storage/CoverageSubset.java
@@ -37,7 +37,7 @@ import org.apache.sis.internal.util.UnmodifiableArrayList;
* arguments of {@link GridCoverageResource#read(GridGeometry, int...)} method.
*
* @author Martin Desruisseaux (Geomatys)
- * @version 1.1
+ * @version 1.2
* @since 1.1
* @module
*/
@@ -153,6 +153,24 @@ final class CoverageSubset extends AbstractGridResource {
}
/**
+ * Returns an indication about when the "physical" loading of raster data
will happen.
+ * This information is fetched to the wrapped resource doing the actual
real operations.
+ */
+ @Override
+ public RasterLoadingStrategy getLoadingStrategy() {
+ return source.getLoadingStrategy();
+ }
+
+ /**
+ * Sets the preferred strategy about when to do the "physical" loading of
raster data.
+ * This information is forwarded to the wrapped resource doing the actual
real operations.
+ */
+ @Override
+ public boolean setLoadingStrategy(final RasterLoadingStrategy strategy) {
+ return source.setLoadingStrategy(strategy);
+ }
+
+ /**
* Loads a subset of the grid coverage represented by this resource.
* The domain to be read by the resource is computed as below:
* <ul>
