Skip to content

Commit

Permalink
[INLONG-10204][Manager] Kafka sink supports automatic allocation of s…
Browse files Browse the repository at this point in the history
…ort standalone cluster (#10207)

Co-authored-by: castorqin <qhj00725@qq.com>
  • Loading branch information
castorqin and castorqin committed May 13, 2024
1 parent f91542b commit 1fa6426
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
import org.apache.inlong.manager.common.util.Preconditions;
import org.apache.inlong.manager.pojo.sink.SinkInfo;
import org.apache.inlong.manager.pojo.sink.kafka.KafkaSinkDTO;
import org.apache.inlong.manager.service.resource.sink.SinkResourceOperator;
import org.apache.inlong.manager.service.resource.sink.AbstractStandaloneSinkResourceOperator;
import org.apache.inlong.manager.service.sink.StreamSinkService;

import org.apache.kafka.clients.admin.Admin;
Expand All @@ -47,7 +47,7 @@
* Kafka resource operator for creating Kafka topic
*/
@Service
public class KafkaResourceOperator implements SinkResourceOperator {
public class KafkaResourceOperator extends AbstractStandaloneSinkResourceOperator {

private static final Logger LOGGER = LoggerFactory.getLogger(KafkaResourceOperator.class);

Expand Down Expand Up @@ -76,7 +76,6 @@ public void createSinkResource(SinkInfo sinkInfo) {
new NewTopic(topicName, Optional.of(partitionNum), Optional.empty())));
result.values().get(topicName).get();
}

sinkService.updateStatus(sinkInfo.getId(), SinkStatus.CONFIG_SUCCESSFUL.getCode(),
"create kafka topic success");
LOGGER.info("success to create kafka topic [{}] for sinkInfo={}", topicName, sinkInfo);
Expand All @@ -85,6 +84,7 @@ public void createSinkResource(SinkInfo sinkInfo) {
sinkService.updateStatus(sinkInfo.getId(), SinkStatus.CONFIG_FAILED.getCode(), e.getMessage());
throw new WorkflowException("create kafka topic failed, reason: " + e.getMessage());
}
this.assignCluster(sinkInfo);
}

/**
Expand Down

0 comments on commit 1fa6426

Please sign in to comment.