diff options
author | Surendra Reddy Katam <SK00524980@techmahindra.com> | 2018-04-03 18:43:56 +0530 |
---|---|---|
committer | Surendra Reddy Katam <SK00524980@techmahindra.com> | 2018-04-03 18:43:56 +0530 |
commit | 04fcb5d8366ec866c1417d1b8063ca4e4aaf5765 (patch) | |
tree | 625a6affc5775445dfa6b7b39c8b9baeef9ee801 | |
parent | f873b47b16a6bb019ffe708fca60df01dd171ef5 (diff) |
Sonar critical issues
Define and throw a dedicated exception instead of using a generic one
Sonar Link:
https://sonar.onap.org/project/issues?assignees=surendraReddy&id=org.onap.dmaap.messagerouter.dmaapclient%3AdmaapClient&open=AV4-MbQa32hFUzlqc5Zc&resolved=false&severities=CRITICAL
Location:
src/main/java/com/att/nsa/mr/client/impl/MRBatchPublisher.java
Line No-78
Change-Id: Ic9fa6e30b450ea1400be0cf8de237eeefdaeb34f
Issue-ID: DMAAP-383
Signed-off-by: Surendra Reddy Katam <SK00524980@techmahindra.com>
-rw-r--r-- | src/main/java/com/att/nsa/mr/client/impl/MRBatchPublisher.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/att/nsa/mr/client/impl/MRBatchPublisher.java b/src/main/java/com/att/nsa/mr/client/impl/MRBatchPublisher.java index 1310986..eb39e8f 100644 --- a/src/main/java/com/att/nsa/mr/client/impl/MRBatchPublisher.java +++ b/src/main/java/com/att/nsa/mr/client/impl/MRBatchPublisher.java @@ -75,7 +75,7 @@ public class MRBatchPublisher implements MRBatchingPublisher try { fSender = new Sender ( baseUrls, topic, maxBatchSize, maxAgeMs, compress ); } catch (MalformedURLException e) { - throw new RuntimeException(e); + throw new IllegalArgumentException(e); } // FIXME: this strategy needs an overhaul -- why not just run a thread that knows how to wait for |