Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Can't start the pulsar connector #1238

Closed
2 tasks done
fengyongshe opened this issue Sep 8, 2022 · 5 comments
Closed
2 tasks done

[Bug] Can't start the pulsar connector #1238

fengyongshe opened this issue Sep 8, 2022 · 5 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@fengyongshe
Copy link
Contributor

fengyongshe commented Sep 8, 2022

Search before asking

  • I had searched in the issues and found no similar issues.

Environment

Linux

EventMesh version

master

What happened

  1. After gradle dist & zip , there is no pulsar connector in dir:./plugin/connector , so as other connectors(pravega,knative.etc)
  2. set connector.plugin.type=pulsar And start runtime, Can't start connector with exception:

Caused by: java.lang.ClassNotFoundException: org.apache.eventmesh.connector.rocketmq.producer.PulsarProducerImpl
at java.net.URLClassLoader.findClass(URLClassLoader.java:381) ~[?:1.8.0_161]
at java.lang.ClassLoader.loadClass(ClassLoader.java:424) ~[?:1.8.0_161]
at java.lang.ClassLoader.loadClass(ClassLoader.java:357) ~[?:1.8.0_161]
at org.apache.eventmesh.spi.loader.JarExtensionClassLoader.lambda$loadResources$1(JarExtensionClassLoader.java:133) ~[eventmesh-spi-1.6.0-release.jar:1.6.0-release]
... 16 more

It is caused by the wrong spi settings in the gradle.properties and META-INF.eventmesh for EventMesh-connector-plugin
image

image

Should't use rocketmq tag in pulsar connector .

How to reproduce

  1. ./gradlew clean dist
  2. ./gradlew installPlugin
  3. bash dist/bin/start.sh //mv pulsar connector to plugin
  4. see dist/logs/eventmesh.out

Debug logs

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!
@fengyongshe fengyongshe added the bug Something isn't working label Sep 8, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2022

Welcome to the Apache EventMesh (incubating) community!!
We are glad that you are contributing by opening this issue. :D

Please make sure to include all the relevant context.
We will be here shortly.

If you are interested in contributing to our project, please let us know!
You can check out our contributing guide on contributing to EventMesh.

Want to get closer to the community?

WeChat Group:
wechat_qr

Mailing Lists:

Name Description Subscribe Unsubscribe Archive
Users User support and questions mailing list Subscribe Unsubscribe Mail Archives
Development Development related discussions Subscribe Unsubscribe Mail Archives
Commits All commits to repositories Subscribe Unsubscribe Mail Archives

@pchengma
Copy link
Member

pchengma commented Sep 9, 2022

Hi @fengyongshe,

Thanks for your question. I reproduced according to the above four steps but did not find the class loading bug for Knative and Pravega connectors. Knative and Pravega connector plugins can be installed in dist/plugin folder after ./gradlew installPlugin and the EventMesh server can be started fine after bash dist/bin/start.sh.

The class loading configuration for the Pulsar connector is in development. I think you can feel free to submit a PR to help fix the related configuration files.

@fengyongshe
Copy link
Contributor Author

I'm willing to accomplish the pulsar connector,Please assign the work to me. @xwm1992 @pchengma

@qqeasonchen
Copy link
Contributor

@fengyongshe sure.

fengyongshe pushed a commit to fengyongshe/incubator-eventmesh that referenced this issue Sep 15, 2022
Motivation

Complete the pulsar connector plugin

Modifications
* Fix the spi wrong settings in resource
* Add Pulsar Consumer for eventmesh connector
* Optimize the ClientConfiguration to cope with service addr

Documentation
fengyongshe pushed a commit to fengyongshe/incubator-eventmesh that referenced this issue Sep 15, 2022
Motivation

fix checkstyle

Modifications

Documentation
fengyongshe pushed a commit to fengyongshe/incubator-eventmesh that referenced this issue Sep 16, 2022
Motivation
Fix ClientConfigurationTest Checkstyle error

Modifications

Documentation
fengyongshe pushed a commit to fengyongshe/incubator-eventmesh that referenced this issue Sep 16, 2022
Motivation
resolve the dependencies jar lost for cloudevent json

Modifications

Documentation
fengyongshe pushed a commit to fengyongshe/incubator-eventmesh that referenced this issue Sep 19, 2022
Motivation
Add Unit Test

Modifications

Documentation
fengyongshe pushed a commit to fengyongshe/incubator-eventmesh that referenced this issue Sep 19, 2022
Motivation
Add License header

Modifications

Documentation
@fengyongshe
Copy link
Contributor Author

Split the issue to different sub-issues for prs manageable

@xwm1992 xwm1992 added this to the 1.7.0 milestone Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants