Refactor CsarUtil::getFromCassandra 89/106989/2
authorFrancis Toth <francis.toth@yoppworks.com>
Fri, 1 May 2020 15:59:31 +0000 (11:59 -0400)
committerOfir Sonsino <ofir.sonsino@intl.att.com>
Tue, 5 May 2020 08:15:05 +0000 (08:15 +0000)
Signed-off-by: Francis Toth <francis.toth@yoppworks.com>
Change-Id: I510d1d7fb4f4a7d7152fd5ea6d4d5ff61deea23a
Issue-ID: SDC-2812

catalog-be/src/main/java/org/openecomp/sdc/be/tosca/CsarUtils.java

index 47a8478..bb388fe 100644 (file)
@@ -652,17 +652,12 @@ public class CsarUtils {
     }
 
     private Either<byte[], ActionStatus> getFromCassandra(String cassandraId) {
-        Either<DAOArtifactData, CassandraOperationStatus> artifactResponse = artifactCassandraDao.getArtifact(cassandraId);
-
-        if (artifactResponse.isRight()) {
-            log.debug("Failed to fetch artifact from Cassandra by id {} error {} ", cassandraId, artifactResponse.right().value());
-
-            StorageOperationStatus storageStatus = DaoStatusConverter.convertCassandraStatusToStorageStatus(artifactResponse.right().value());
-            ActionStatus convertedFromStorageResponse = componentsUtils.convertFromStorageResponse(storageStatus);
-            return Either.right(convertedFromStorageResponse);
-        }
-        DAOArtifactData artifactData = artifactResponse.left().value();
-        return Either.left(artifactData.getDataAsArray());
+        return artifactCassandraDao.getArtifact(cassandraId)
+            .right().map(cos -> {
+                log.debug("Failed to fetch artifact from Cassandra by id {} error {} ", cassandraId, cos);
+                StorageOperationStatus storageStatus = DaoStatusConverter.convertCassandraStatusToStorageStatus(cos);
+                return componentsUtils.convertFromStorageResponse(storageStatus);
+            }).left().map(DAOArtifactData::getDataAsArray);
     }
 
     private String createCsarBlock0(String metaFileVersion, String toscaConformanceLevel) {