You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by he...@apache.org on 2022/09/09 06:13:25 UTC

[inlong] branch release-1.3.0 updated: [INLONG-5830][DataProxy] Fix required parameter topic exception (#5831)

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

healchow pushed a commit to branch release-1.3.0
in repository https://gitbox.apache.org/repos/asf/inlong.git


The following commit(s) were added to refs/heads/release-1.3.0 by this push:
     new 1fe9d7ae6 [INLONG-5830][DataProxy] Fix required parameter topic exception (#5831)
1fe9d7ae6 is described below

commit 1fe9d7ae6c1e080fe5898bea1e48d1db402ac042
Author: xuesongxs <54...@users.noreply.github.com>
AuthorDate: Fri Sep 9 10:27:32 2022 +0800

    [INLONG-5830][DataProxy] Fix required parameter topic exception (#5831)
---
 .../src/main/java/org/apache/inlong/dataproxy/http/HttpBaseSource.java | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/inlong-dataproxy/dataproxy-source/src/main/java/org/apache/inlong/dataproxy/http/HttpBaseSource.java b/inlong-dataproxy/dataproxy-source/src/main/java/org/apache/inlong/dataproxy/http/HttpBaseSource.java
index 4b563d5ca..84e45e2fd 100644
--- a/inlong-dataproxy/dataproxy-source/src/main/java/org/apache/inlong/dataproxy/http/HttpBaseSource.java
+++ b/inlong-dataproxy/dataproxy-source/src/main/java/org/apache/inlong/dataproxy/http/HttpBaseSource.java
@@ -117,10 +117,7 @@ public class HttpBaseSource extends AbstractSource implements EventDrivenSource,
 
         topic = context.getString(ConfigConstants.TOPIC);
         attr = context.getString(ConfigConstants.ATTR);
-        Configurables.ensureRequiredNonNull(context, ConfigConstants.TOPIC, ConfigConstants.ATTR);
 
-        topic = topic.trim();
-        Preconditions.checkArgument(!topic.isEmpty(), "topic is empty");
         attr = attr.trim();
         Preconditions.checkArgument(!attr.isEmpty(), "attr is empty");