From: Surendra Reddy Katam Date: Thu, 10 Jan 2019 05:26:12 +0000 (+0530) Subject: Sonar majior issues X-Git-Tag: 1.1.16~14 X-Git-Url: https://gerrit.onap.org/r/gitweb?p=dmaap%2Fmessagerouter%2Fmsgrtr.git;a=commitdiff_plain;h=203a6de85ecb784531747612bc61fe30f6bfe351 Sonar majior issues This block of commented-out lines of code should be removed Sonar Link: https://sonar.onap.org/project/issues?assignees=surendraReddy&id=org.onap.dmaap.messagerouter.msgrtr%3Amsgrtr&open=AWYBFCTAligbnTmNJUX3&resolved=false&severities=MAJOR Location: src/main/java/org/onap/dmaap/dmf/mr/service/impl/EventsServiceImpl.java Line No-L640 L653 L684 L701 L730 L859 Change-Id: I7369cfb186a2c906359d53b7c92b035039952280 Issue-ID: DMAAP-956 Signed-off-by: Surendra Reddy Katam --- diff --git a/src/main/java/org/onap/dmaap/dmf/mr/service/impl/EventsServiceImpl.java b/src/main/java/org/onap/dmaap/dmf/mr/service/impl/EventsServiceImpl.java index 387b667..2f436ec 100644 --- a/src/main/java/org/onap/dmaap/dmf/mr/service/impl/EventsServiceImpl.java +++ b/src/main/java/org/onap/dmaap/dmf/mr/service/impl/EventsServiceImpl.java @@ -637,7 +637,7 @@ public class EventsServiceImpl implements EventsService { // final KeyedMessage data = new // KeyedMessage(topic, m.getKey(), - // kms.add(data); + final ProducerRecord data = new ProducerRecord(topic, m.getKey(), m.getMessage()); @@ -650,7 +650,7 @@ public class EventsServiceImpl implements EventsService { + batchId + "]"); try { // ctx.getConfigReader().getfPublisher().sendBatchMessage(topic, - // kms); + ctx.getConfigReader().getfPublisher().sendBatchMessageNew(topic, pms); // transactionLogs(batch); for (message msg : batch) { @@ -681,7 +681,7 @@ public class EventsServiceImpl implements EventsService { metricsSet.publishTick(sizeNow); publishBatchCount = sizeNow; count += sizeNow; - // batchId++; + String endTime = sdf.format(new Date()); LOG.info("Batch End Details:[serverIp=" + ctx.getRequest().getLocalAddr() + ",Batch End Id=" + batchId + ",Batch Total=" + publishBatchCount + ",Batch Start Time=" + startTime @@ -698,9 +698,9 @@ public class EventsServiceImpl implements EventsService { + batchId + "]"); try { // ctx.getConfigReader().getfPublisher().sendBatchMessage(topic, - // kms); + ctx.getConfigReader().getfPublisher().sendBatchMessageNew(topic, pms); - // transactionLogs(batch); + for (message msg : batch) { LogDetails logDetails = msg.getLogDetails(); LOG.info("Publisher Log Details : " + logDetails.getPublisherLogDetails()); @@ -727,7 +727,7 @@ public class EventsServiceImpl implements EventsService { pms.clear(); metricsSet.publishTick(sizeNow); count += sizeNow; - // batchId++; + String endTime = sdf.format(new Date()); publishBatchCount = sizeNow; LOG.info("Batch End Details:[serverIp=" + ctx.getRequest().getLocalAddr() + ",Batch End Id=" + batchId @@ -855,12 +855,12 @@ public class EventsServiceImpl implements EventsService { return logDetails; } - /* - * public String getMetricsTopic() { return metricsTopic; } - * - * public void setMetricsTopic(String metricsTopic) { this.metricsTopic = - * metricsTopic; } - */ + + + + + +