log warn when not ObjectId is found

telefonicaid · Oct 8, 2024 · 36dd84c · 36dd84c
1 parent f9932ff
commit 36dd84c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/cygnus-common/src/main/java/com/telefonica/iot/cygnus/backends/arcgis/model/Feature.java b/cygnus-common/src/main/java/com/telefonica/iot/cygnus/backends/arcgis/model/Feature.java
@@ -276,6 +276,7 @@ public Integer getObjectId() throws ArcgisException {
             }
         }
         if (objectId.equals(-1)) {
+            LOGGER.warn("Cant find " + GisAttributeType.OID + " in Feature Object.");
             throw new ArcgisException("Cant find " + GisAttributeType.OID + " in Feature Object.");
         } else {
             return objectId;
@@ -305,7 +306,7 @@ public void setObjectId(Integer objectId) throws ArcgisException {
             }
         } catch (Exception e) {
             throw new ArcgisException(
-                    "Error setting OBJECTID for feature " + this.toString() + " - Error: " + e);
+                    "Error setting OBJECTID for feature " + this.toString() + " with value " + objectId + " - Error: " + e);
         }
     }
 

diff --git a/...src/main/java/com/telefonica/iot/cygnus/backends/arcgis/restutils/ArcgisFeatureTable.java b/...src/main/java/com/telefonica/iot/cygnus/backends/arcgis/restutils/ArcgisFeatureTable.java
@@ -398,6 +398,7 @@ protected void splitFeatureListIfExists(List<Feature> featureArray,
                     if (featureId.equalsIgnoreCase(serverFeatureId)) {
                         found = true;
                         Integer oid = serverFeature.getObjectId();
+                        LOGGER.debug("retrieved ObjectId: " + oid + " from feature " + serverFeatureId);
                         feature.setObjectId(oid);
                         existentFeatures.add(feature);
                     }