From: Dushyant Singh Thakur <dt00503496@techmahindra.com>
Date: Wed, 14 Feb 2018 09:36:03 +0000 (+0530)
Subject: sonar critical for error handling
X-Git-Tag: v1.1.2~9^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=f2f7d77c3a9655fe0dbfcd2c7b0c991395e2c3de;p=dmaap%2Fmessagerouter%2Fmsgrtr.git

sonar critical for error handling

Either log or rethrow this exception
Sonar Link:
https://sonar.onap.org/project/issues?assignees=dthakur&id=org.onap.dmaap.messagerouter.msgrtr%3Amsgrtr&resolved=false&severities=CRITICAL
Location:
src/main/java/com/att/nsa/cambria/beans/DMaaPKafkaConsumerFactory.java
Line No-185

Change-Id: I6df9f1afead4e846b1eb831ae4f5c00bd7e9011c
Issue-ID: DMAAP-232
Signed-off-by: Dushyant Singh Thakur <dt00503496@techmahindra.com>
---

diff --git a/src/main/java/com/att/nsa/cambria/beans/DMaaPKafkaConsumerFactory.java b/src/main/java/com/att/nsa/cambria/beans/DMaaPKafkaConsumerFactory.java
index 28d48fa..59dc9bf 100644
--- a/src/main/java/com/att/nsa/cambria/beans/DMaaPKafkaConsumerFactory.java
+++ b/src/main/java/com/att/nsa/cambria/beans/DMaaPKafkaConsumerFactory.java
@@ -183,7 +183,8 @@ public class DMaaPKafkaConsumerFactory implements ConsumerFactory {
 							kc);
 				}
 			} catch (org.I0Itec.zkclient.exception.ZkTimeoutException x) {
-				log.warn("Kafka consumer couldn't connect to ZK.");
+				log.error("Exception find at getConsumerFor(String topic, String consumerGroupName,\r\n" + 
+						"			String consumerId, int timeoutMs) : " + x);
 				throw new UnavailableException("Couldn't connect to ZK.");
 			} catch (KafkaConsumerCacheException e) {
 				log.warn("Failed to cache consumer (this may have performance implications): "