[
https://issues.apache.org/jira/browse/TUBEMQ-96?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Guo Jiwei resolved TUBEMQ-96.
-----------------------------
Resolution: Fixed
> Fix typo & use IllegalArgumentException
> ---------------------------------------
>
> Key: TUBEMQ-96
> URL: https://issues.apache.org/jira/browse/TUBEMQ-96
> Project: Apache TubeMQ
> Issue Type: Improvement
> Reporter: Guo Jiwei
> Assignee: Guo Jiwei
> Priority: Low
> Labels: pull-request-available
>
> 1. Fix typo
> validConsumerGroupParmeter -> validConsumerGroupParameter
> pushIsListenerWaitTimeoutRollBack -> pushListenerWaitTimeoutRollBack
> pushIsListenerThrowedRollBack -> pushListenerThrowedRollBack
> 2. Use IllegalArgumentException
> In ConsumerConfig#validConsumerGroupParameter
> {code:java}
> private void validConsumerGroupParameter(String consumerGroup) throws
> Exception {
> if (TStringUtils.isBlank(consumerGroup)) {
> throw new Exception("Illegal parameter: consumerGroup is Blank!");
> }
> String tmpConsumerGroup = String.valueOf(consumerGroup).trim();
> if (tmpConsumerGroup.length() >
> TBaseConstants.META_MAX_GROUPNAME_LENGTH) {
> throw new Exception(new StringBuilder(512)
> .append("Illegal parameter: the max length of
> consumerGroup is ")
> .append(TBaseConstants.META_MAX_GROUPNAME_LENGTH)
> .append(" characters").toString());
> }
> if (!tmpConsumerGroup.matches(TBaseConstants.META_TMP_GROUP_VALUE)) {
> throw new Exception(new StringBuilder(512)
> .append("Illegal parameter: the value of consumerGroup")
> .append(" must begin with a letter, ")
> .append("can only contain characters,numbers,hyphen,and
> underscores").toString());
> }
> }
> {code}
> will change to throw IllegalArgumentException
> {code:java}
> private void validConsumerGroupParameter(String consumerGroup) throws
> Exception {
> if (TStringUtils.isBlank(consumerGroup)) {
> throw new IllegalArgumentException("Illegal parameter:
> consumerGroup is Blank!");
> }
> String tmpConsumerGroup = String.valueOf(consumerGroup).trim();
> if (tmpConsumerGroup.length() >
> TBaseConstants.META_MAX_GROUPNAME_LENGTH) {
> throw new IllegalArgumentException(new StringBuilder(512)
> .append("Illegal parameter: the max length of
> consumerGroup is ")
> .append(TBaseConstants.META_MAX_GROUPNAME_LENGTH)
> .append(" characters").toString());
> }
> if (!tmpConsumerGroup.matches(TBaseConstants.META_TMP_GROUP_VALUE)) {
> throw new IllegalArgumentException(new StringBuilder(512)
> .append("Illegal parameter: the value of consumerGroup")
> .append(" must begin with a letter, ")
> .append("can only contain characters,numbers,hyphen,and
> underscores").toString());
> }
> }
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
