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

[Improve][Tubemq] Missing parameter #8812

Closed
2 tasks done
SCUTking opened this issue Aug 29, 2023 · 4 comments · Fixed by #8921
Closed
2 tasks done

[Improve][Tubemq] Missing parameter #8812

SCUTking opened this issue Aug 29, 2023 · 4 comments · Fixed by #8921

Comments

@SCUTking
Copy link
Contributor

Description

The parameter of the AddTopicReq class is missing memCacheMsgSizeInMB compared to the HTTP API parameter.
image
image

InLong Component

InLong TubeMQ

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct

@github-actions
Copy link

Hello @SCUTking, thank you for opening your first issue in InLong 🧡 We will respond as soon as possible ⏳
If this is a bug report, please provide screenshots or error logs for us to reproduce your issue, so we can do our best to fix it.
If you have any questions in the meantime, you can also ask us on the InLong Discussions 🔍

@gosonzhang
Copy link
Contributor

It does not fully map TubeMQ's API parameters, which feels like this problem should be caused by the omission of the original implementer.

@SCUTking, would you like to take some time to fix it?

@SCUTking
Copy link
Contributor Author

I am very happy to fix it.

@gosonzhang
Copy link
Contributor

@SCUTking, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment