Skip to content

Commit 3d5f95f

Browse files
Make logging of unknown fields in OME metadata less severe
This warning fired e.g. when reading https://livingobjects.ebi.ac.uk/idr/zarr/v0.5/idr0033A/BR00109990_C2.zarr/0 Since, this log can actually be ignored, I propose to lower the severity of the log.
1 parent d5552e7 commit 3d5f95f

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

src/main/java/dev/zarr/zarrjava/experimental/ome/OmeObjectMappers.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ static ObjectMapper makeV3Mapper() {
3232

3333
private static final class UnknownOmePropertyWarningHandler extends DeserializationProblemHandler {
3434
private static final Logger LOGGER = Logger.getLogger(UnknownOmePropertyWarningHandler.class.getName());
35-
private static final Set<String> WARNED_FIELDS = ConcurrentHashMap.newKeySet();
35+
private static final Set<String> UNKNOWN_FIELDS = ConcurrentHashMap.newKeySet();
3636

3737
@Override
3838
public boolean handleUnknownProperty(
@@ -46,8 +46,8 @@ public boolean handleUnknownProperty(
4646
? ((Class<?>) beanOrClass).getName()
4747
: beanOrClass.getClass().getName();
4848
String key = target + "#" + propertyName;
49-
if (WARNED_FIELDS.add(key)) {
50-
LOGGER.warning(
49+
if (UNKNOWN_FIELDS.add(key)) {
50+
LOGGER.fine(
5151
"Ignoring unknown OME metadata field '" + propertyName + "' for " + target);
5252
}
5353
p.skipChildren();

0 commit comments

Comments
 (0)