[DMAAP-CLIENT] First sonar issues review part2

[dmaap/messagerouter/dmaapclient.git] / src / main / java / org / onap / dmaap / mr / client / MRConsumer.java

diff --git a/src/main/java/org/onap/dmaap/mr/client/MRConsumer.java b/src/main/java/org/onap/dmaap/mr/client/MRConsumer.java
index b2f7563..8196a0a 100644 (file)
--- a/src/main/java/org/onap/dmaap/mr/client/MRConsumer.java
+++ b/src/main/java/org/onap/dmaap/mr/client/MRConsumer.java
@@ -4,11 +4,13 @@
  *  ================================================================================
  *  Copyright © 2017 AT&T Intellectual Property. All rights reserved.
  *  ================================================================================
+ *  Modifications Copyright © 2021 Orange.
+ *  ================================================================================
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
  *  You may obtain a copy of the License at
  *        http://www.apache.org/licenses/LICENSE-2.0
- *  
+ *
  *  Unless required by applicable law or agreed to in writing, software
  *  distributed under the License is distributed on an "AS IS" BASIS,
  *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -17,38 +19,37 @@
  *  ============LICENSE_END=========================================================
  *
  *  ECOMP is a trademark and service mark of AT&T Intellectual Property.
- *  
+ *
  *******************************************************************************/
+
 package org.onap.dmaap.mr.client;
 
 import java.io.IOException;
-
 import org.onap.dmaap.mr.client.response.MRConsumerResponse;
 
-public interface MRConsumer extends MRClient
-{
-       /**
-        * Fetch a set of messages. The consumer's timeout and message limit are used if set in the constructor call. 
+public interface MRConsumer extends MRClient {
+    /**
+     * Fetch a set of messages. The consumer's timeout and message limit are used if set in the constructor call.
+     *
+     * @return a set of messages
+     * @throws IOException
+     */
+    Iterable<String> fetch() throws IOException, Exception;
+
+    /**
+     * Fetch a set of messages with an explicit timeout and limit for this call. These values
+     * override any set in the constructor call.
+     *
+     * @param timeoutMs The amount of time in milliseconds that the server should keep the connection
+     *                  open while waiting for message traffic. Use -1 for default timeout (controlled on the server-side).
+     * @param limit     A limit on the number of messages returned in a single call. Use -1 for no limit.
+     * @return a set messages
+     * @throws IOException if there's a problem connecting to the server
+     */
+    Iterable<String> fetch(int timeoutMs, int limit) throws IOException, Exception;
+
+    MRConsumerResponse fetchWithReturnConsumerResponse();
 
-        * @return a set of messages
-        * @throws IOException
-        */
-       Iterable<String> fetch ()  throws IOException, Exception;
 
-       /**
-        * Fetch a set of messages with an explicit timeout and limit for this call. These values
-        * override any set in the constructor call.
-        * 
-        * @param timeoutMs     The amount of time in milliseconds that the server should keep the connection
-        * open while waiting for message traffic. Use -1 for default timeout (controlled on the server-side).
-        * @param limit A limit on the number of messages returned in a single call. Use -1 for no limit.
-        * @return a set messages
-        * @throws IOException if there's a problem connecting to the server
-        */
-       Iterable<String> fetch ( int timeoutMs, int limit )  throws IOException, Exception;
-       
-       MRConsumerResponse fetchWithReturnConsumerResponse ();
-       
-       
-       MRConsumerResponse fetchWithReturnConsumerResponse ( int timeoutMs, int limit );
+    MRConsumerResponse fetchWithReturnConsumerResponse(int timeoutMs, int limit);
 }