You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by do...@apache.org on 2022/02/19 10:24:48 UTC

[incubator-inlong] branch master updated: [INLONG-2596][TubeMQ] fix param error and add override annotation (#2595)

This is an automated email from the ASF dual-hosted git repository.

dockerzhang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-inlong.git


The following commit(s) were added to refs/heads/master by this push:
     new 9f3a951  [INLONG-2596][TubeMQ] fix param error and add override annotation (#2595)
9f3a951 is described below

commit 9f3a9519ca200c35584c6e8dce55e4eca334cc53
Author: yimaixinchen <yi...@163.com>
AuthorDate: Sat Feb 19 18:24:41 2022 +0800

    [INLONG-2596][TubeMQ] fix param error and add override annotation (#2595)
---
 .../org/apache/inlong/tubemq/client/common/QueryMetaResult.java     | 2 ++
 .../java/org/apache/inlong/tubemq/client/consumer/PartitionExt.java | 6 +++---
 inlong-tubemq/tubemq-manager/pom.xml                                | 2 +-
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/common/QueryMetaResult.java b/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/common/QueryMetaResult.java
index 996fd48..f3452eb 100644
--- a/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/common/QueryMetaResult.java
+++ b/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/common/QueryMetaResult.java
@@ -37,10 +37,12 @@ public class QueryMetaResult extends RetValue {
         super(errCode, errInfo);
     }
 
+    @Override
     public void setFailResult(int errCode, final String errMsg) {
         super.setFailResult(errCode, errMsg);
     }
 
+    @Override
     public void setFailResult(final String errMsg) {
         super.setFailResult(errMsg);
     }
diff --git a/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/consumer/PartitionExt.java b/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/consumer/PartitionExt.java
index 5006495..5d38fc1 100644
--- a/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/consumer/PartitionExt.java
+++ b/inlong-tubemq/tubemq-client/src/main/java/org/apache/inlong/tubemq/client/consumer/PartitionExt.java
@@ -80,12 +80,12 @@ public class PartitionExt extends Partition {
      * @param isReqEscLimit   if the rsplimitDlt is ignored in current consume
      * @param rsplimitDlt     max offset of the current consume
      * @param lastDataDlt     offset of the last data fetch
-     * @param isRRequireSlow  if the server requires slow down
+     * @param isRequireSlow  if the server requires slow down
      * @return message size per minute
      */
     public long procConsumeResult(boolean isFilterConsume, int reqProcType, int errCode,
                                   int msgSize, boolean isReqEscLimit, long rsplimitDlt,
-                                  long lastDataDlt, boolean isRRequireSlow) {
+                                  long lastDataDlt, boolean isRequireSlow) {
         if (lastDataDlt >= 0) {
             this.lastDataRdDlt = lastDataDlt;
         }
@@ -121,7 +121,7 @@ public class PartitionExt extends Partition {
             }
             if (errCode == TErrCodeConstants.SUCCESS) {
                 if (isFilterConsume && filterCtrlItem.getFreqLtInMs() >= 0) {
-                    if (isRRequireSlow) {
+                    if (isRequireSlow) {
                         return this.filterCtrlItem.getZeroCnt();
                     } else {
                         return this.filterCtrlItem.getFreqLtInMs();
diff --git a/inlong-tubemq/tubemq-manager/pom.xml b/inlong-tubemq/tubemq-manager/pom.xml
index e81a32d..683b67d 100644
--- a/inlong-tubemq/tubemq-manager/pom.xml
+++ b/inlong-tubemq/tubemq-manager/pom.xml
@@ -176,7 +176,7 @@
                     </descriptors>
                     <archive>
                         <manifest>
-                            <mainClass>com.cyy.MvnPackageApplication</mainClass>
+                            <mainClass>org.apache.inlong.tubemq.manager.TubeMQManager</mainClass>
                         </manifest>
                     </archive>
                 </configuration>