From bce8bc8383c3f19b64b360a941ec1b1503791c85 Mon Sep 17 00:00:00 2001 From: Ravi Geda Date: Thu, 5 Jul 2018 11:11:55 +0100 Subject: Fix asynchronous patch requests Async flow uses gson to de-serialize the event to a ChampObject which is causing inconsistencies for timestamp property values. Fix is to use jackson which the sync flow also uses. Change-Id: Ic766d34a8c04b9d076d7cadb53cb9dcb180186b6 Issue-ID: AAI-1363 Signed-off-by: Ravi Geda --- .../champ/async/ChampAsyncRequestProcessor.java | 2 +- .../org/onap/champ/event/GraphEventVertex.java | 25 ++++++++++++++++++---- 2 files changed, 22 insertions(+), 5 deletions(-) (limited to 'champ-service/src/main/java') diff --git a/champ-service/src/main/java/org/onap/champ/async/ChampAsyncRequestProcessor.java b/champ-service/src/main/java/org/onap/champ/async/ChampAsyncRequestProcessor.java index 4966287..a52127e 100644 --- a/champ-service/src/main/java/org/onap/champ/async/ChampAsyncRequestProcessor.java +++ b/champ-service/src/main/java/org/onap/champ/async/ChampAsyncRequestProcessor.java @@ -187,7 +187,7 @@ public class ChampAsyncRequestProcessor extends TimerTask { case UPDATE: event.setVertex(GraphEventVertex.fromChampObject( - champDataService.replaceObject(event.getVertex().toChampObject(), + champDataService.replaceObject(event.getVertex().toChampObject(event.getVertex().toJson()), event.getVertex().getId(), Optional.ofNullable(transaction)), event.getVertex().getModelVersion())); break; diff --git a/champ-service/src/main/java/org/onap/champ/event/GraphEventVertex.java b/champ-service/src/main/java/org/onap/champ/event/GraphEventVertex.java index cc4f493..866c8ae 100644 --- a/champ-service/src/main/java/org/onap/champ/event/GraphEventVertex.java +++ b/champ-service/src/main/java/org/onap/champ/event/GraphEventVertex.java @@ -20,13 +20,14 @@ */ package org.onap.champ.event; +import java.io.IOException; import java.util.Map; - import javax.ws.rs.core.Response.Status; - import org.onap.aai.champcore.model.ChampObject; +import org.onap.champ.entity.ChampObjectDeserializer; import org.onap.champ.exception.ChampServiceException; - +import com.fasterxml.jackson.databind.ObjectMapper; +import com.fasterxml.jackson.databind.module.SimpleModule; import com.google.gson.Gson; import com.google.gson.GsonBuilder; import com.google.gson.JsonElement; @@ -160,7 +161,7 @@ public class GraphEventVertex { return graphEventVertex; } - + public ChampObject toChampObject() { ChampObject.Builder builder = new ChampObject.Builder(this.getType()); @@ -180,4 +181,20 @@ public class GraphEventVertex { } + /** + * Uses jackson api to convert the json string into a ChampObject. + * + * @param json + * @return + * @throws IOException + */ + // Added this method as gson conversion of timestamp properties(create ts and modified ts) differs from the jackson conversion causing failures. + public ChampObject toChampObject(String json) throws IOException { + ObjectMapper mapper = new ObjectMapper(); + SimpleModule module = new SimpleModule(); + module.addDeserializer(ChampObject.class, new ChampObjectDeserializer()); + mapper.registerModule(module); + return mapper.readValue(json, ChampObject.class); + } + } -- cgit 1.2.3-korg