From: Congcong Peng Date: Sat, 24 Feb 2018 08:53:03 +0000 (+0800) Subject: replace Jackson with fastJson X-Git-Tag: 1.2.0~22 X-Git-Url: https://gerrit.onap.org/r/gitweb?p=holmes%2Fdsa.git;a=commitdiff_plain;h=be988c78605a1e90e13829db6505d12cd26ea1b4 replace Jackson with fastJson Issue-ID: HOLMES-115 Change-Id: I1dbcd7c0061a930c85afe460614f16c2e6d5514c Signed-off-by: Congcong Peng --- diff --git a/dmaap-dsa/pom.xml b/dmaap-dsa/pom.xml index a999176..f0c7bd9 100644 --- a/dmaap-dsa/pom.xml +++ b/dmaap-dsa/pom.xml @@ -29,6 +29,13 @@ jar + + + com.alibaba + fastjson + 1.1.23 + + org.onap.msb.java-sdk msb-java-sdk @@ -71,6 +78,14 @@ log4j-over-slf4j org.slf4j + + com.fasterxml.jackson.core + jackson-databind + + + com.fasterxml.jackson.core + jackson-core + @@ -147,12 +162,6 @@ json-simple 1.1.1 - - net.sf.json-lib - json-lib - 2.4 - jdk15 - org.glassfish.jersey.core jersey-server @@ -177,6 +186,12 @@ com.google.guava guava 19.0 + + + com.fasterxml.jackson.core + jackson-core + + diff --git a/dmaap-dsa/src/main/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtil.java b/dmaap-dsa/src/main/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtil.java index 5dffc7d..be30153 100644 --- a/dmaap-dsa/src/main/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtil.java +++ b/dmaap-dsa/src/main/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtil.java @@ -15,95 +15,96 @@ */ package org.onap.holmes.dsa.dmaappolling; -import com.fasterxml.jackson.databind.JsonNode; -import com.fasterxml.jackson.databind.ObjectMapper; -import java.io.IOException; -import java.util.ArrayList; -import java.util.List; +import com.alibaba.fastjson.JSON; +import com.alibaba.fastjson.JSONArray; +import com.alibaba.fastjson.JSONObject; import org.jvnet.hk2.annotations.Service; import org.onap.holmes.common.api.stat.AlarmAdditionalField; import org.onap.holmes.common.api.stat.VesAlarm; +import java.io.IOException; +import java.util.ArrayList; +import java.util.List; + @Service public class DMaaPResponseUtil { public VesAlarm convertJsonToVesAlarm(String responseJson) throws IOException { - ObjectMapper mapper = new ObjectMapper(); - JsonNode jsonNode = mapper.readTree(responseJson); - VesAlarm vesAlarm = new VesAlarm(); + JSONObject jsonNode = JSON.parseObject(responseJson); - JsonNode eventJson = jsonNode.get("event"); + VesAlarm vesAlarm = new VesAlarm(); - JsonNode commonEventHeaderJson = eventJson.get("commonEventHeader"); + JSONObject eventJson = JSON.parseObject(jsonNode.get("event") +""); + JSONObject commonEventHeaderJson = JSON.parseObject(eventJson.get("commonEventHeader") +""); convertCommonEventHeaderJsonToEvent(commonEventHeaderJson, vesAlarm); - JsonNode faultFieldsJson = eventJson.get("faultFields"); + JSONObject faultFieldsJson = JSON.parseObject(eventJson.get("faultFields") +""); convertFaultFieldsJsonToEvent(faultFieldsJson, vesAlarm); return vesAlarm; } - private void convertCommonEventHeaderJsonToEvent(JsonNode commonEventHeaderJson, - VesAlarm vesAlarm) { - vesAlarm.setDomain(commonEventHeaderJson.get("domain").asText()); - vesAlarm.setEventId(commonEventHeaderJson.get("eventId").asText()); - vesAlarm.setEventName(commonEventHeaderJson.get("eventName").asText()); + private void convertCommonEventHeaderJsonToEvent(JSONObject commonEventHeaderJson, + VesAlarm vesAlarm) { + vesAlarm.setDomain((String) commonEventHeaderJson.get("domain")); + vesAlarm.setEventId((String) commonEventHeaderJson.get("eventId")); + vesAlarm.setEventName((String) commonEventHeaderJson.get("eventName")); vesAlarm.setAlarmIsCleared(vesAlarm.getEventName().endsWith("Cleared") ? 1 : 0); vesAlarm.setEventType(getTextElementByNode(commonEventHeaderJson, "eventType")); vesAlarm.setInternalHeaderFields( getTextElementByNode(commonEventHeaderJson, "internalHeaderFields")); - vesAlarm.setLastEpochMicrosec(commonEventHeaderJson.get("lastEpochMicrosec").asLong()); + vesAlarm.setLastEpochMicrosec(commonEventHeaderJson.getLong("lastEpochMicrosec")); vesAlarm.setNfcNamingCode(getTextElementByNode(commonEventHeaderJson, "nfcNamingCode")); vesAlarm.setNfNamingCode(getTextElementByNode(commonEventHeaderJson, "nfNamingCode")); - vesAlarm.setPriority(commonEventHeaderJson.get("priority").asText()); + vesAlarm.setPriority((String) commonEventHeaderJson.get("priority")); vesAlarm.setReportingEntityId( getTextElementByNode(commonEventHeaderJson, "reportingEntityId")); - vesAlarm.setReportingEntityName(commonEventHeaderJson.get("reportingEntityName").asText()); - vesAlarm.setSequence(commonEventHeaderJson.get("sequence").asInt()); + vesAlarm.setReportingEntityName( (String) commonEventHeaderJson.get("reportingEntityName")); + vesAlarm.setSequence((Integer) commonEventHeaderJson.get("sequence")); vesAlarm.setSourceId(getTextElementByNode(commonEventHeaderJson, "sourceId")); - vesAlarm.setSourceName(commonEventHeaderJson.get("sourceName").asText()); - vesAlarm.setStartEpochMicrosec(commonEventHeaderJson.get("startEpochMicrosec").asLong()); - vesAlarm.setVersion(commonEventHeaderJson.get("version").asLong()); + vesAlarm.setSourceName( (String) commonEventHeaderJson.get("sourceName")); + vesAlarm.setStartEpochMicrosec(commonEventHeaderJson.getLong("startEpochMicrosec")); + vesAlarm.setVersion(commonEventHeaderJson.getLong("version")); } - private void convertFaultFieldsJsonToEvent(JsonNode faultFieldsJson, VesAlarm vesAlarm) { + private void convertFaultFieldsJsonToEvent(JSONObject faultFieldsJson, VesAlarm vesAlarm) { vesAlarm.setAlarmAdditionalInformation(getListElementByNode(faultFieldsJson, "alarmAdditionalInformation")); - vesAlarm.setAlarmCondition(faultFieldsJson.get("alarmCondition").asText()); + vesAlarm.setAlarmCondition(faultFieldsJson.getString("alarmCondition")); vesAlarm.setAlarmInterfaceA(getTextElementByNode(faultFieldsJson, "alarmInterfaceA")); vesAlarm.setEventCategory(getTextElementByNode(faultFieldsJson,"eventCategory")); - vesAlarm.setEventSeverity(faultFieldsJson.get("eventSeverity").asText()); - vesAlarm.setEventSourceType(faultFieldsJson.get("eventSourceType").asText()); - vesAlarm.setFaultFieldsVersion(faultFieldsJson.get("faultFieldsVersion").asLong()); - vesAlarm.setSpecificProblem(faultFieldsJson.get("specificProblem").asText()); - vesAlarm.setVfStatus(faultFieldsJson.get("vfStatus").asText()); + vesAlarm.setEventSeverity(faultFieldsJson.getString("eventSeverity")); + vesAlarm.setEventSourceType(faultFieldsJson.getString("eventSourceType")); + vesAlarm.setFaultFieldsVersion(faultFieldsJson.getLong("faultFieldsVersion")); + vesAlarm.setSpecificProblem(faultFieldsJson.getString("specificProblem")); + vesAlarm.setVfStatus(faultFieldsJson.getString("vfStatus")); } - private String getTextElementByNode(JsonNode jsonNode,String name){ - if(jsonNode.has(name)){ - return jsonNode.get(name).asText(); + private String getTextElementByNode(JSONObject jsonNode,String name){ + if(jsonNode.get(name) != null){ + return jsonNode.getString(name); } return null; } - private Long getLongElementByNode(JsonNode jsonNode, String name) { - if(jsonNode.has(name)){ - return jsonNode.get(name).asLong(); + private Long getLongElementByNode(JSONObject jsonNode, String name) { + if(jsonNode.get(name) != null){ + return jsonNode.getLong(name); } return null; } - private List getListElementByNode(JsonNode jsonNode, String name){ + private List getListElementByNode(JSONObject jsonNode, String name){ List alarms = new ArrayList(); - if (jsonNode.has(name)) { - JsonNode alarmAdditionalInformations = jsonNode.get(name); - if (alarmAdditionalInformations.isArray()) { - alarmAdditionalInformations.forEach(alarm -> { - if(alarm.has("name") && alarm.has("value")) { - AlarmAdditionalField field = new AlarmAdditionalField(); - field.setName(getTextElementByNode(alarm, "name")); - field.setValue(getTextElementByNode(alarm, "value")); - alarms.add(field); - } - }); + if (jsonNode.get(name) != null) { + JSONArray alarmAdditionalInformations = jsonNode.getJSONArray(name); + for (int i = 0; i < alarmAdditionalInformations.size(); i++) { + JSONObject jsonObject = alarmAdditionalInformations.getJSONObject(i); + if (jsonObject.get("name") != null + && jsonObject.get("value") != null) { + AlarmAdditionalField field = new AlarmAdditionalField(); + field.setName(getTextElementByNode(jsonObject, "name")); + field.setValue(getTextElementByNode(jsonObject, "value")); + alarms.add(field); + } } } return alarms; diff --git a/dmaap-dsa/src/test/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtilTest.java b/dmaap-dsa/src/test/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtilTest.java index aacbee3..26fba51 100644 --- a/dmaap-dsa/src/test/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtilTest.java +++ b/dmaap-dsa/src/test/java/org/onap/holmes/dsa/dmaappolling/DMaaPResponseUtilTest.java @@ -21,6 +21,8 @@ import static org.junit.Assert.assertThat; import java.io.IOException; import java.util.ArrayList; import java.util.List; + +import com.alibaba.fastjson.JSONException; import org.junit.Before; import org.junit.Rule; import org.junit.Test; @@ -161,7 +163,7 @@ public class DMaaPResponseUtilTest { @Test public void testDMaaPResponseUtil_input_illegal() throws Exception { String json = "***"; - thrown.expect(IOException.class); + thrown.expect(JSONException.class); dMaaPResponseUtil.convertJsonToVesAlarm(json); }