Fix potential null pointers

Change-Id: I309010480012cc66ac8d44d6be65f5897174ec3f
Issue-ID: AAI-1430
Signed-off-by: bogumil_zebek <bogumil.zebek@nokia.com>

Merge "Fix equals and hashCode methods"

Merge "Fix sonar violation"

Fix equals and hashCode methods

Change-Id: I3e3ebcd1ba29bc20a41c138ec071f137ccd6cf18
Issue-ID: AAI-1375
Signed-off-by: Bogumil Zebek <bogumil.zebek@nokia.com>

Merge "Fix handling of InterruptedException in AAI-Champ"

Fix sonar violation

Fix potential places where NullPointerException could be thrown.

Change-Id: I84c3dc7b5c1a1f39dcf7d432adac8c507fbc5e3b
Issue-ID: AAI-1369
Signed-off-by: Bogumil Zebek <bogumil.zebek@nokia.com>

Merge "change order in class for java convention"

Fix handling of InterruptedException in AAI-Champ

Added thread interrupts

Change-Id: Ic5b7eef71354efd02736e88a1e6716aa84169898
Issue-ID: AAI-1367
Signed-off-by: Wiktor Garbarek <wiktor.garbarek@nokia.com>

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 <GRaviK@amdocs.com>

change order in class for java convention

Change-Id: I45b681aae0bd09909a5ae1bb6f546ae32d1a8821
Issue-ID: AAI-1346
Signed-off-by: jczestochowska <justyna.czestochowska@nokia.com>

Merge "Refactor of NoOpTinkerPopTransaction"

Refactor of NoOpTinkerPopTransaction

Removed unused logger; added method annotation

Change-Id: Ie1bc4754042189cf231e08445c53f34e8efb06ad
Issue-ID: AAI-1247
Signed-off-by: Wiktor Garbarek <wiktor.garbarek@nokia.com>

Merge "Refactor code smell AbstractValidatingChampGraph"

Fix serialization of the "key" property

The introduction of a payload envelope to the update notification event
issued by champ-core caused the incorrect serialization of the "key"
property as a JSON object instead of a String.
This caused Spike to fail when parsing the notification event from
Champ as it expects "key" to be a String.

A previous fix attempt incorrectly addressed this problem. As the issue
originated from the inconsistent use of both Jackson and Gson JSON
libraries with the envelope using Gson but the contained objects
using Jackson annotations. This fix primarily involves updates to
champ-core to ensure that the envelope generation uses Jackson
instead of Gson and respects the original Jackson serialization of
the "key" property as a String.

Change-Id: I2a12732c9ff3970c3db9de5f0039304e68cb3556
Issue-ID: AAI-1243
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Refactor code smell AbstractValidatingChampGraph

Change-Id: I6e76af608a326866de54acd8c5398a04267143ae
Issue-ID: AAI-1237
Signed-off-by: jczestochowska <justyna.czestochowska@nokia.com>

Fix serialization of update notification payload

Fix Gson serialization of ChampEventEnvelope:
- serialize key as primitive.
  Currently output as object ({}) with a "value" property.
  This is due to Optional object declaration and misalignment between
  Gson serializer and Jackson annotations in ChampObject
- serialize transactionId as transaction-id
- serialize dbTransactionId as db-transaction-id

Change-Id: I3a9824ad376ca8189ba31cc9442cb42bb06a9d70
Issue-ID: AAI-1218
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Add validation of request headers

Enforces presence of X-FromAppId and X-TransactionId headers
in REST requests.

Change-Id: I539e863049e4d5a985d9e952ee7dcbf3fd97f7b3
Issue-ID: AAI-1194
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Return ETag in response header

This update generates a unique hash for the payload and returns it
as an etag header.

Change-Id: I471dc1e74e8096d4fdb4f4db7f22e08ef4842ace
Issue-ID: AAI-1208
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Fix Fortify scan vulnerabilities

Change-Id: I9a402ca1bb1755cf5ff9fc04eba781ed7f2b4a1c
Issue-ID: AAI-1193
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Update payload format for update notification

The update notification event is now aligned with the format of the
response event.  Both response and update notification events issued
by Champ now include top level header and body properties.

Change-Id: I00f9971cdbab4944def66c25f5939d5cc4de71bd
Issue-ID: AAI-1195
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Ignore reserved query params

Change-Id: I3eadc1e238a6f73d2dee1b6ced9b1d6033ded85e
Issue-ID: AAI-1203
Signed-off-by: Shwetank Dave <shwetank.dave@amdocs.com>

Make champ run as root

Change-Id: I769540fd4982641e92fdcad87df91d532bbed565
Issue-ID: AAI-1175
Signed-off-by: Daniel Silverthorn <daniel.silverthorn@amdocs.com>

Merge "Dynamically determine image architecture"

Dynamically determine image architecture

The image architecture is currently hardcoded, this patch removes
that hardcoded value to allow for ARM64 builds to go through.

Change-Id: I576949e2f09261007f79efc238cde249426c77e6
Issue-ID: CIMAN-154
Signed-off-by: Mohammed Naser <mnaser@vexxhost.com>

Pin jnr-unixsocket for Docker plugin to 0.13

The current verison of jnr-unixsocket is 0.12 from the Docker
plugin however in order to be able the images for ARM64, it needs
the newer version which has support for it.

There is work upstream to switch the version to 0.18[0] however
it seems that this is still pending, so we can pin it for now until
that merges.

[0]: https://github.com/spotify/docker-maven-plugin/pull/379

Change-Id: Ieb1b5c6e43d3dc1425b2ab63f2fd161e452e5a57
Issue-ID: CIMAN-154
Signed-off-by: Mohammed Naser <mnaser@vexxhost.com>

Merge "Remove broken not used code"

Upversion for Casablanca

Issue-ID: AAI-1169
Change-Id: I640156e35b5de1b509fac5abaabeddd6917c00bb
Signed-off-by: Jimmy Forsyth <jf2512@att.com>

Upversion poms and version.properties to 1.2.3

Change-Id: I2956bf79f9aaedfa3f3cd5fc7caf6f794e7f02a1
Issue-ID: AAI-1154
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Upversion to 1.2.2 for Beijing release

Change-Id: I3113bdda31f2a702e1a8f7a5446fefba1f843db6
Issue-ID: AAI-1153
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Organise pom structure into corrected hierarchy

All child modules are now properly grouped under each parent.
This should allow the Jenkins update version plugin to update
the version of all child modules and allow the staging deployment
of the champ-service releasable artifact.

Change-Id: I6ecae622654e736f65691d9fa4357d27e606c801
Issue-ID: AAI-1146
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Update champ-service pom to force staging

This is an attempt to force the autorelease staging.

Change-Id: I203c9e27fc41d4d4a61ba9a25a07479d50e162c9
Issue-ID: AAI-1146
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Update champ-service pom to correct groupId

This is a further attempt to fix the release-version-java-daily job.

Change-Id: I0df7889eaf040baea5a2d8791fc9d5f2361b5690
Issue-ID: AAI-1146
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Update poms to upload champ-service jar

The intention is to additionally resolve the build of the
release-version-java-daily job.

Change-Id: I39af216e187245a845cc1f4921a02c21b4728884
Issue-ID: AAI-1146
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Upversion to 1.2.1-SNAPSHOT

Issue-ID: AAI-1138
Change-Id: Ia5f8ce5b51f994b4418c673b5ee3bffe049c8446
Signed-off-by: Jimmy Forsyth <jf2512@att.com>

Remove duplicate dependency

Issue-ID: AAI-1141
Change-Id: I8d470492d4e55f82b70f82ee8d54ea4bde662c83
Signed-off-by: Jimmy Forsyth <jf2512@att.com>

Update version to 1.2.1-SNAPSHOT

Change-Id: I46cad4df0a03b5114c08efad2b04c2a19573a4b5
Issue-ID: AAI-1138
Signed-off-by: Ravi Geda <GRaviK@amdocs.com>

Remove broken not used code

Change-Id: I3e34778705ffca18f9646f9cb888446995e743f7
Issue-ID: AAI-1129
Signed-off-by: Bogumil Zebek <bogumil.zebek@nokia.com>

Address security vulnerabilities (fix 2)

Fixes previously missed dependencies:
- jackson-databind: version 2.8.11.1
- httpclient: version 4.5.3
- jackson-mapper: version 1.9.2

Change-Id: Iae2013bf164a90e910ec1d2cd4ad1185a01bacad
Issue-ID: AAI-1117
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Address security vulnerabilities

This includes version upgrades for:
- logback-classic, logback-core
- commons-collections
- hadoop-common
- hbase-client: settled on version 1.0.2 as container would not start
                with later versions
- httpclient
- netty, netty-all
- zookeeper
- jackson-core
Also should resolve presence of flux-examples.

Change-Id: Ifb55f5d6676a9971d1d9a46c695dc78eb1b99843
Issue-ID: AAI-1117
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Prevent deployment of champ-service.jar

The deployment of champ-service.jar is causing the merge job to fail.
The champ service jar is required for docker only.

Change-Id: Ia944aae5cf19857e2890be4251534f59bbedd027
Issue-ID: AAI-1119
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Update Dockerfile and remove redundant AJSC files

Change-Id: Ibf166602929901d18d0ee24c8d5505e4489000fa
Issue-ID: AAI-1114
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Replace AJSC with Spring Boot 1.5.12.RELEASE

Updates include:
Remove AJSC references
Use PropertiesLoader (loader.path) to allow loading external janus or titan deps
Fix dependency conflicts between janus/titan deps and Spring Boot deps
Use Jetty container
Use Jersey registration of Champ REST interface

Change-Id: Ic33b886b0b039cce0366c0f0910cc5a3fe5bfd9b
Issue-ID: AAI-1041
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Upgrade to janus 0.2.0

Change-Id: I47de44fed3113817096127eb4e849ede259eaad8
Issue-ID: AAI-1082
Signed-off-by: Daniel Silverthorn <daniel.silverthorn@amdocs.com>

Correct INFO.yaml to pass YAML Lint

Change-Id: Iae5012a4c1248002fb02af4caefee7f0834223fb
Issue-ID: AAI-1022
Signed-off-by: Eric Debeau <eric.debeau@orange.com>

Fix jdk tools system dependency in champ-lib

Update jdk tools systempath definition to use java.home property.

Issue-ID: AAI-1017

Change-Id: Iec7ad7bb42908b9844df875f74c4d9a65244afe9
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Update to consume and publish events in new format

The new format includes:
- the graph request/response encapsulated in a body property
- new event header with details such as timestamp, request-id,
  event-type

Issue-ID: AAI-960

Change-Id: Ib84ddd54352ca95c3968d2d2936f6348951c2d2c
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Merge "Catch IllegalArgumentException for error message"

Catch IllegalArgumentException for error message

Also had to upversion event client library

Change-Id: I4dbbf8f35630d93cca9e5d8971e7fde8a65b0811
Issue-ID: AAI-945
Signed-off-by: Daniel Silverthorn <daniel.silverthorn@amdocs.com>

Update module poms to skip maven deploy

The rationale for this change is that the only deployable artifact is
the champ microservice as a docker component. The intention is to
allow the failing Jenkins merge job to complete.

Issue-ID: AAI-927

Change-Id: Ib73470697cd79d12b8d1d47df9afa930f4768d8f
Signed-off-by: Michael Arrastia <MArrasti@amdocs.com>

Merge "Fix docker image name"

Fix docker image name

Correct the docker image name

Change-Id: Idb0ac585468e65f54db97517ed010a78b01d87b4
Issue-ID: AAI-933
Signed-off-by: Ravi Geda <gravik@amdocs.com>

Add maven deploy and nexus staging plugins

Fix merge job which is failing with authorisation error.

Issue-ID: AAI-927

Change-Id: Ibbf969de8d4d9d8238f8385a0c3d074887ef56dc
Signed-off-by: MichaelArrastia <MArrasti@amdocs.com>

Support maven deploy and docker build targets

This commit principally includes the following pom updates:
1. Add distributionManagement to fix jobs that run maven deploy.
2. Update way in which champ-service gets the graph-deps. This
change specifically supports the docker build.

Issue-ID: AAI-927

Change-Id: I5920de3c010b810a802f95ac82da89ea3989e7d9
Signed-off-by: MichaelArrastia <MArrasti@amdocs.com>

Update license date and text

Update license date and disable license plugi:n

Issue-ID: AAI-862

Change-Id: Ic46fee4917bde48298acfc1d76eef9083986113e
Signed-off-by: gabriel.sanchez <gabriel.sanchez@amdocs.com>

Tidy pom's following event-client version update

Removed ability to override the version of the event-client

Issue-ID: AAI-873

Change-Id: I49c76cca0ac6da49964df277467f57a463b65069
Signed-off-by: michaere <michaere@amdocs.com>

Fix name of entry in Sonar

The name of the entry in Sonar was displayed as champ instaed of aai-champ

Change-Id: I5a3edc59592fa13c4273274f52716e09157ee812
Issue-ID: CIMAN-65
Signed-off-by: Gildas Lanilis <gildas.lanilis@huawei.com>

Fix items missed in microservice restructure

1. Updates the champ-service pom to explicitly add a dependency
on the apache httpclient.
2. Corrects the name of the top level champ pom

Issue-ID: AAI-813

Change-Id: I155ebc723861eea8e81e96ba84f182e80c7e42ed
Signed-off-by: MichaelArrastia <MArrasti@amdocs.com>

Add Spotify and License plugins

The champ-service module pom requires additional plugins:
1. Spotify: to allow ONAP Jenkins docker job to complete
2. License: to generate license headers in Java files

Issue-ID: AAI-850

Change-Id: Ie5b82d36773c32cb1bba428c0effb8b04fbb7859
Signed-off-by: MichaelArrastia <MArrasti@amdocs.com>

Remove DistributionManagement section from pom

Updates to champ-lib pom also include the removal of scm tags, which were pointing to specific
repositories. This will be handled by the settings.xml and build job

Issue-ID: AAI-848

Change-Id: I3bca4cc05748388fb806b81474d26776bc1b677b
Signed-off-by: michaere <michaere@amdocs.com>

Port champ-microservice project restructure

Includes project restructure and introduction of a parent pom. The
original source folder and core functionality is now held within champ-lib, with champ-service
forming the ajsc microservice from which it injects champ-lib core
functionality.

Issue-ID: AAI-813

Change-Id: I2ce0c4a70e485665276e7955572de23969deb706
Signed-off-by: michaere <michaere@amdocs.com>

Either log or rethrow this exception

Either log or rethrow this exception
AbstractLoggingChampGraph.java:L444

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKDlaLbmlVXUW53&resolved=false
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Ia7db723244de872b0aa7ec6c726e53012949edec
Issue-ID: AAI-825
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Either log or rethrow this exception

Either log or rethrow this exception
AbstractLoggingChampGraph.java:L441

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKDlaLbmlVXUW53&resolved=false
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Ia1b329cd8541fb0fb3dcd29729a57ebf39004fb8
Issue-ID: AAI-811
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Merge "Either log or rethrow this exception"

Either log or rethrow this exception

Either log or rethrow this exception
AbstractLoggingChampGraph.java:L272

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKDlaLbmlVXUW53&resolved=false
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Idda83f60ad5f568a513582c885fb6295a9682f8a
Issue-ID: AAI-810
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Add INFO.yaml file

Add INFO.yaml to list:
- Project description
- Properties
- PTL information
- Meeting information
- Committer information

Change-Id: Ibda05bcba2706a8f70559b7913ea76b52d01df1e
Issue-ID: CIMAN-134
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L116

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: I27f7fcb6b67863ee38e4627f67051b2ad0584c9f
Issue-ID: AAI-784
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L114

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Ie69b744af3e3947a3f856695bf09b213e32a2adb
Issue-ID: AAI-783
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L109

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: I29b98526132420f02de4eae25a4541b04497285f
Issue-ID: AAI-776
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L115

Sonar link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: I1bd5d950d7fd4a682b0e55fd20ba89d002a8a28b
Issue-ID: AAI-786
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Merge "Reorder the modifiers"

Merge "Reorder the modifiers"

Merge "Reorder the modifiers"

Merge "Add the @Override annotation"

Merge "Add the @Override annotation"

Merge "Add the @Override annotation"

Merge "Add the @Override annotation"

Merge "Add the @Override annotation"

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L112

Sonar link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Ifd9e5b595f8cf2d4a259cc6e96e4f0ee1b8ef208
Issue-ID: AAI-785
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L108

Sonar link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Id9b452fd59208629d35310df8a09faf8c7d987e3
Issue-ID: AAI-782
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L103

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: I25f87ac57ca0cd9f5989820dfda368a165c661cb
Issue-ID: AAI-775
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Add the @Override annotation

Add the @Override annotation above this method signature
AbstractLoggingChampGraph.java:L85

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Ifd6f0fbffb797db4d9823fd5c3769e7a098f32cf
Issue-ID: AAI-754
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Add the @Override annotation

Add the @Override annotation above this method signature
AbstractLoggingChampGraph.java:L82

Sonar link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.javaChange-Id: I75f7308ac13cb238176066a1311530259a1d2f94

Change-Id: Ib474d08bb509a82f247f03cd6b1f7837ac26eb45
Issue-ID: AAI-753
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Add the @Override annotation

Add the @Override annotation above this method signature
AbstractLoggingChampGraph.java:L81

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: I4e6808285c6353cc4a2af0687f5bafea3c3ff79a
Issue-ID: AAI-752
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Add the @Override annotation

Add the @Override annotation above this method signature
AbstractLoggingChampGraph.java:L76

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: I94f4d83af7a8458436b907fdf5ab983c4da3e0ba
Issue-ID: AAI-750
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Add the @Override annotation

Add the @Override annotation above this method signature
AbstractLoggingChampGraph.java:L77

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&open=AV7ygfKAlaLbmlVXUW5d
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: I6a3c705ed9152c8ab10749bfda1cceb8b20c3d19
Issue-ID: AAI-751
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Merge "Reorder the modifiers"

Reorder the modifiers

Reorder the modifiers to comply with the Java Language Specification
AbstractLoggingChampGraph.java:L98

Sonar Link : https://sonar.onap.org/project/issues?assignees=PrasadNaidu&id=org.onap.aai%3Achamp&resolved=false
Location : src/main/java/org/onap/aai/champ/event/AbstractLoggingChampGraph.java

Change-Id: Ie2c5195b47231a031851b7e0a819f69725d627fe
Issue-ID: AAI-774
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Merge "Add the @Override annotation"

Add a private constructor

Add a private constructor to hide the implicit public one
ChampGraph.java:L62

Sonar Link : https://sonar.onap.org/component_issues/index?id=org.onap.aai%3Achamp#
Location : src/main/java/org/onap/aai/champ/ChampGraph.java

Change-Id: I402638841541fafc8c7b1b81d27bf2ea53686502
Issue-ID: AAI-744
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Add the @Override annotation

Add the @Override annotation above this method signature
AbstractLoggingChampGraph.java:L74

Sonar Link : https://sonar.onap.org/component_issues/index?id=org.onap.aai%3Achamp#resolved=false|assignees=PrasadNaidu
Location : src/main/java/org/onap/aai/champ/ChampGraph.javaChange-Id: Ifef3f30780de982c898bd2095a14379ef6844a48

Change-Id: I961be806db8afc69e15805563c44a5b95e2b46b5
Issue-ID: AAI-749
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Commented-out lines of code should be removed

This block of commented-out lines of code should be removed
ChampGraph.java:L73

Sonar Link : https://sonar.onap.org/component_issues/index?id=org.onap.aai%3Achamp#assignees=PrasadNaidu
Location : src/main/java/org/onap/aai/champ/ChampGraph.javaChange-Id: Icbd8a40bdf8bc5169bf4de02ea2c13a6c56ea42a

Change-Id: I66e72b1299c19af67e8836e8c2a78f3c4fcd4e2f
Issue-ID: AAI-746
Signed-off-by: PrasadNaidu <RV00493633@techmahindra.com>

Renaming openecomp to onap

Issue-ID: AAI-208
Change-Id: I2635c4ed4f9e5d17b93a8ec2c3a0d82b79f75320
Signed-off-by: Arul.Nambi <arul.nambi@amdocs.com>

Merge "Added @Override annotation above signature"

Added @Override annotation above signature

Fix Major sonar issues in aai/aai-champ module
https://sonar.onap.org/component_issues?id=org.openecomp.aai%3Achamp#resolved=false|severities=MAJOR

Add the "@Override" annotation above this method signature

Issue-Id:AAI-246
Change-Id: I5e9ef77e2e6e5da91e51f75858f7aeb178d2aa02
Signed-off-by: shashikanth.vh <shashikanth.vh@huawei.com>

Fix string literal comparision on left side

Fix Major sonar issues in aai/aai-champ module
https://sonar.onap.org/component_issues?id=org.openecomp.aai%3Achamp#resolved=false|severities=MAJOR

Fixed issue Move string literal on the left side of this string comparison

Issue-Id:AAI-246
Change-Id: I2c714ffca0933f5083e1f37d6ac70d79f018ecdc
Signed-off-by: shashikanth.vh <shashikanth.vh@huawei.com>

Fix the jacoco file reference in pom file

Issue-ID: AAI-306
Change-Id: Iae59ccb7f7625c1b477963daa145a22f31b2660d
Signed-off-by: Venkata Harish K Kajur <vk250x@att.com>

Cleanup project's name in Sonar

The name parameter in the root pom.xml should
match the project name in gerrit to reflect consistency
in Sonar.

Change-Id: I6b00b9cfff8677d89a12f027cefb7380d208e0be
Issue-id: CIMAN-65
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Add name to the top level pom for champ

Issue-ID: AAI-271

Change-Id: I6a6c81fa3cf12e4007cf6edbcb11fc9e5f57c52f
Signed-off-by: Venkata Harish K Kajur <vk250x@att.com>

Merge "Add missing .gitreview file"

Update POM to inherit from oparent

Change-Id: Iaf15716b64ff2b22c59504caf824e9400a98e3f8
Issue-Id: AAI-77
Signed-off-by: Dileep Ranganathan <dileep.ranganathan@intel.com>