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

[Enhancement] LMQ logic queue order maybe wrong #8693

Closed
1 task done
redlsz opened this issue Sep 13, 2024 · 0 comments · Fixed by #8700
Closed
1 task done

[Enhancement] LMQ logic queue order maybe wrong #8693

redlsz opened this issue Sep 13, 2024 · 0 comments · Fixed by #8700

Comments

@redlsz
Copy link
Contributor

redlsz commented Sep 13, 2024

Before Creating the Enhancement Request

  • I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary

image

The INNER_MULTI_QUEUE_OFFSET of LMQ messages may be not continuous. As shown in the screenshot, "1075474,1075481,1075481" is skipped. By checking the commitlog, we found that this offset is occupied by one RETRY message but is not dispatched to LMQ cq in the end.

image

Store error logs when this issue occurs:
image

Motivation

Make sure the LMQ logic queue order is correct.

Describe the Solution You'd Like

When appending commitlog, do not allocate LMQ offset for messages of system topics, be consistent with consume queue dispath logic.

Describe Alternatives You've Considered

/

Additional Context

No response

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

Successfully merging a pull request may close this issue.

1 participant