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

[mqtt-to-kafka] Migrate akka dependencies to pekko #22

Merged
merged 5 commits into from
Dec 17, 2023

Conversation

laglangyue
Copy link
Contributor

@laglangyue laglangyue commented Dec 17, 2023

this pr inclueds:
Using pekko instead of akka in the document and code.
Using pekko-connectors style by https://github.com/apache/incubator-pekko-connectors/blob/main/.scalafmt.conf,because this module uses 4 spaces for indentation.
Test:
test local by docker 24.0.7 on Mint Linux 21
test failed,because missing mqtt container.

@laglangyue
Copy link
Contributor Author

Due to the dependency of this sample on docker-compose.yml, testing is a bit cumbersome and cannot be completed directly by running the main function. I think it will improve the testing of Later

@pjfanning pjfanning merged commit d6aa4d0 into apache:main Dec 17, 2023
12 checks passed
@laglangyue laglangyue deleted the migrate_mqtt_to_kafka branch December 18, 2023 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants