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.

Sign up for GitHub

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

Jump to bottom

fix RocketMQ Source has no namesrv set error #35

Merged
merged 7 commits into from
Aug 23, 2021
Merged

fix RocketMQ Source has no namesrv set error #35

merged 7 commits into from
Aug 23, 2021

Conversation

francisoliverlee
Copy link
Member

some fixes

  • RocketMQ Source can't work cause of no namesrv set and update its tests
  • add some examples about how to use RocketMQ-Streams

tigerweili and others added 3 commits August 23, 2021 01:11
fix bugs
60c38b2 
1. leak of interface api for compile error
2. add namesrv for RocketMQ Source
fix RocketMQ 4.5.2 and 4.8.0 's differences in RocketMQSource
e606e40
Merge pull request #1 from apache/main
f651ed0 
merge from apache main
@francisoliverlee francisoliverlee linked an issue Aug 22, 2021 that may be closed by this pull request
[bug] rocketmq source can't work cause of no namesrv is set #34
Closed
tiger lee and others added 4 commits August 23, 2021 10:17
Merge pull request #2 from apache/main
26ceaf7 
merge from apache master
add apache file headers
06eeb45
Merge branch 'main' of https://github.com/francisoliverlee/rocketmq-s…
865528b 
…treams into main

* 'main' of https://github.com/francisoliverlee/rocketmq-streams:
  add license header
  feat(doc) add quickstart link
  feat(code style) exclude ci files in apache rat check
  feat(doc) change Chinese doc file path
  update readme.md
  Update README.md
  can not compile
fix code check fail: not ovride removeSplit() and addNewSplit()
c03b1b9
@duhenglucky duhenglucky merged commit a4d234a into apache:main Aug 23, 2021
ni-ze pushed a commit to ni-ze/rocketmq-streams that referenced this pull request Oct 7, 2022
@duhenglucky
Merge pull request apache#35 from francisoliverlee/main
b1a0541 
fix RocketMQ Source has no namesrv set error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bug] rocketmq source can't work cause of no namesrv is set
2 participants
@francisoliverlee @duhenglucky