From 3d5f95f636e19a501c16163b2c22559733f56e93 Mon Sep 17 00:00:00 2001 From: Stefan Hahmann Date: Thu, 7 May 2026 13:43:05 +0200 Subject: [PATCH] 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. --- .../zarr/zarrjava/experimental/ome/OmeObjectMappers.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/dev/zarr/zarrjava/experimental/ome/OmeObjectMappers.java b/src/main/java/dev/zarr/zarrjava/experimental/ome/OmeObjectMappers.java index 9f1c7c3f..977fda7f 100644 --- a/src/main/java/dev/zarr/zarrjava/experimental/ome/OmeObjectMappers.java +++ b/src/main/java/dev/zarr/zarrjava/experimental/ome/OmeObjectMappers.java @@ -32,7 +32,7 @@ static ObjectMapper makeV3Mapper() { private static final class UnknownOmePropertyWarningHandler extends DeserializationProblemHandler { private static final Logger LOGGER = Logger.getLogger(UnknownOmePropertyWarningHandler.class.getName()); - private static final Set WARNED_FIELDS = ConcurrentHashMap.newKeySet(); + private static final Set UNKNOWN_FIELDS = ConcurrentHashMap.newKeySet(); @Override public boolean handleUnknownProperty( @@ -46,8 +46,8 @@ public boolean handleUnknownProperty( ? ((Class) beanOrClass).getName() : beanOrClass.getClass().getName(); String key = target + "#" + propertyName; - if (WARNED_FIELDS.add(key)) { - LOGGER.warning( + if (UNKNOWN_FIELDS.add(key)) { + LOGGER.fine( "Ignoring unknown OME metadata field '" + propertyName + "' for " + target); } p.skipChildren();