diff --git a/client/src/main/java/org/apache/rocketmq/client/impl/MQClientAPIImpl.java b/client/src/main/java/org/apache/rocketmq/client/impl/MQClientAPIImpl.java index e7e805d1738..be70d9f5fae 100644 --- a/client/src/main/java/org/apache/rocketmq/client/impl/MQClientAPIImpl.java +++ b/client/src/main/java/org/apache/rocketmq/client/impl/MQClientAPIImpl.java @@ -300,7 +300,7 @@ public void createTopic(final String addr, final String defaultTopic, final Topi public void createPlainAccessConfig(final String addr, final PlainAccessConfig plainAccessConfig, final long timeoutMillis) - throws RemotingException, MQBrokerException, InterruptedException, MQClientException { + throws RemotingException, InterruptedException, MQClientException { CreateAccessConfigRequestHeader requestHeader = new CreateAccessConfigRequestHeader(); requestHeader.setAccessKey(plainAccessConfig.getAccessKey()); requestHeader.setSecretKey(plainAccessConfig.getSecretKey()); @@ -328,7 +328,7 @@ public void createPlainAccessConfig(final String addr, final PlainAccessConfig p } public void deleteAccessConfig(final String addr, final String accessKey, final long timeoutMillis) - throws RemotingException, MQBrokerException, InterruptedException, MQClientException { + throws RemotingException, InterruptedException, MQClientException { DeleteAccessConfigRequestHeader requestHeader = new DeleteAccessConfigRequestHeader(); requestHeader.setAccessKey(accessKey); @@ -349,7 +349,7 @@ public void deleteAccessConfig(final String addr, final String accessKey, final } public void updateGlobalWhiteAddrsConfig(final String addr, final String globalWhiteAddrs, final long timeoutMillis) - throws RemotingException, MQBrokerException, InterruptedException, MQClientException { + throws RemotingException, InterruptedException, MQClientException { UpdateGlobalWhiteAddrsConfigRequestHeader requestHeader = new UpdateGlobalWhiteAddrsConfigRequestHeader(); requestHeader.setGlobalWhiteAddrs(globalWhiteAddrs); @@ -404,7 +404,7 @@ public ClusterAclVersionInfo getBrokerClusterAclInfo(final String addr, } public AclConfig getBrokerClusterConfig(final String addr, - final long timeoutMillis) throws RemotingCommandException, InterruptedException, RemotingTimeoutException, + final long timeoutMillis) throws InterruptedException, RemotingTimeoutException, RemotingSendRequestException, RemotingConnectException, MQBrokerException { RemotingCommand request = RemotingCommand.createRequestCommand(RequestCode.GET_BROKER_CLUSTER_ACL_CONFIG, null); @@ -528,7 +528,7 @@ private void sendMessageAsync( final AtomicInteger times, final SendMessageContext context, final DefaultMQProducerImpl producer - ) throws InterruptedException, RemotingException { + ) { final long beginStartTime = System.currentTimeMillis(); try { this.remotingClient.invokeAsync(addr, request, timeoutMillis, new InvokeCallback() { @@ -621,22 +621,10 @@ private void onExceptionImpl(final String brokerName, } String addr = instance.findBrokerAddressInPublish(retryBrokerName); log.warn("async send msg by retry {} times. topic={}, brokerAddr={}, brokerName={}", tmp, msg.getTopic(), addr, - retryBrokerName, e); - try { - request.setOpaque(RemotingCommand.createNewRequestId()); - sendMessageAsync(addr, retryBrokerName, msg, timeoutMillis, request, sendCallback, topicPublishInfo, instance, + retryBrokerName, e); + request.setOpaque(RemotingCommand.createNewRequestId()); + sendMessageAsync(addr, retryBrokerName, msg, timeoutMillis, request, sendCallback, topicPublishInfo, instance, timesTotal, curTimes, context, producer); - } catch (InterruptedException e1) { - onExceptionImpl(retryBrokerName, msg, timeoutMillis, request, sendCallback, topicPublishInfo, instance, timesTotal, curTimes, e1, - context, false, producer); - } catch (RemotingTooMuchRequestException e1) { - onExceptionImpl(retryBrokerName, msg, timeoutMillis, request, sendCallback, topicPublishInfo, instance, timesTotal, curTimes, e1, - context, false, producer); - } catch (RemotingException e1) { - producer.updateFaultItem(brokerName, 3000, true); - onExceptionImpl(retryBrokerName, msg, timeoutMillis, request, sendCallback, topicPublishInfo, instance, timesTotal, curTimes, e1, - context, true, producer); - } } else { if (context != null) { @@ -1073,7 +1061,7 @@ public void endTransactionOneway( final EndTransactionRequestHeader requestHeader, final String remark, final long timeoutMillis - ) throws RemotingException, MQBrokerException, InterruptedException { + ) throws RemotingException, InterruptedException { RemotingCommand request = RemotingCommand.createRequestCommand(RequestCode.END_TRANSACTION, requestHeader); request.setRemark(remark); @@ -2103,8 +2091,7 @@ public BrokerStatsData viewBrokerStatsData(String brokerAddr, String statsName, } public Set getClusterList(String topic, - long timeoutMillis) throws MQClientException, RemotingConnectException, - RemotingSendRequestException, RemotingTimeoutException, InterruptedException { + long timeoutMillis) { return Collections.EMPTY_SET; } @@ -2170,8 +2157,7 @@ public TopicConfigSerializeWrapper getAllTopicConfig(final String addr, } public void updateNameServerConfig(final Properties properties, final List nameServers, long timeoutMillis) - throws UnsupportedEncodingException, - MQBrokerException, InterruptedException, RemotingTimeoutException, RemotingSendRequestException, + throws UnsupportedEncodingException, InterruptedException, RemotingTimeoutException, RemotingSendRequestException, RemotingConnectException, MQClientException { String str = MixAll.properties2String(properties); if (str == null || str.length() < 1) { @@ -2281,7 +2267,7 @@ public void checkClientInBroker(final String brokerAddr, final String consumerGr } public boolean resumeCheckHalfMessage(final String addr, String msgId, - final long timeoutMillis) throws RemotingException, MQClientException, InterruptedException { + final long timeoutMillis) throws RemotingException, InterruptedException { ResumeCheckHalfMessageRequestHeader requestHeader = new ResumeCheckHalfMessageRequestHeader(); requestHeader.setMsgId(msgId); diff --git a/srvutil/src/main/java/org/apache/rocketmq/srvutil/FileWatchService.java b/srvutil/src/main/java/org/apache/rocketmq/srvutil/FileWatchService.java index 111cedea792..c2aaf9620e8 100644 --- a/srvutil/src/main/java/org/apache/rocketmq/srvutil/FileWatchService.java +++ b/srvutil/src/main/java/org/apache/rocketmq/srvutil/FileWatchService.java @@ -23,7 +23,6 @@ import java.nio.file.Path; import java.nio.file.Paths; import java.security.MessageDigest; -import java.security.NoSuchAlgorithmException; import java.util.ArrayList; import java.util.List; import org.apache.commons.lang3.StringUtils; @@ -89,7 +88,7 @@ public void run() { log.info(this.getServiceName() + " service end"); } - private String hash(String filePath) throws IOException, NoSuchAlgorithmException { + private String hash(String filePath) throws IOException { Path path = Paths.get(filePath); md.update(Files.readAllBytes(path)); byte[] hash = md.digest();