Skip to content

Commit

Permalink
Fix issues identified when executing the workflow
Browse files Browse the repository at this point in the history
- DataConversions should always call the wrapped methods
- Only one log provider should be included in the project
- Use MonotonicDataMap for coordinates and references
  • Loading branch information
bchapuis committed Sep 16, 2024
1 parent 7502fb1 commit f5accab
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ public DataSource getDataSource(Object database) {
}

public Map<Long, Coordinate> getCoordinateMap() throws IOException {
return DataConversions.asMap(getMemoryAlignedDataMap("coordinates", new LonLatDataType()));
return DataConversions.asMap(getMonotonicDataMap("coordinates", new LonLatDataType()));
}

public Map<Long, List<Long>> getReferenceMap() throws IOException {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -270,13 +270,7 @@ public V put(K key, V value) {

@Override
public V get(Object key) {
if (map instanceof MemoryAlignedDataMap) {
return map.get(key);
} else if (map instanceof IndexedDataMap) {
return map.get(key);
} else {
return super.get(key);
}
return map.get(key);
}

@Override
Expand Down
6 changes: 6 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -237,6 +237,12 @@ limitations under the License.
<groupId>mil.nga.geopackage</groupId>
<artifactId>geopackage</artifactId>
<version>${version.lib.geopackage}</version>
<exclusions>
<exclusion>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-nop</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>net.ripe.ipresource</groupId>
Expand Down

0 comments on commit f5accab

Please sign in to comment.