Skip to content

Commit

Permalink
#1157: Fix for S3BlobStore bulk delete (without listeners)
Browse files Browse the repository at this point in the history
(failing since a133c44, test ignored since 855ed09)
  • Loading branch information
JeroenVu authored and github-actions[bot] committed Oct 16, 2023
1 parent 4e059be commit 5e62392
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ public String parametersMetadataPrefix(final String layerName) {
* @return the key prefix up to the coordinates (i.e. {@code
* "<prefix>/<layer>/<gridset>/<format>/<parametersId>"})
*/
public String coordinatesPrefix(TileRange obj, boolean endWithSlah) {
public String coordinatesPrefix(TileRange obj, boolean endWithSlash) {
checkNotNull(obj.getLayerName());
checkNotNull(obj.getGridSetId());
checkNotNull(obj.getMimeType());
Expand All @@ -215,7 +215,7 @@ public String coordinatesPrefix(TileRange obj, boolean endWithSlah) {
}
String shortFormat = mimeType.getFileExtension(); // png, png8, png24, etc

String key = join(endWithSlah, prefix, layer, gridset, shortFormat, parametersId);
String key = join(endWithSlash, prefix, layer, gridset, shortFormat, parametersId);
return key;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ public KeyVersion apply(long[] loc) {
@Override
public boolean delete(final TileRange tileRange) throws StorageException {

final String coordsPrefix = keyBuilder.coordinatesPrefix(tileRange, false);
final String coordsPrefix = keyBuilder.coordinatesPrefix(tileRange, true);
if (!s3Ops.prefixExists(coordsPrefix)) {
return false;
}
Expand Down

0 comments on commit 5e62392

Please sign in to comment.