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

Use transactions python consumer #2908

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hubertdeng123
Copy link
Member

Seems like this is not used in prod, and not extensively tested and may be the reason why the Stats page is not properly showing events that are accepted/filtered/dropped.

Copy link
Collaborator

@aldy505 aldy505 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm gonna give it a try in a few hours.

@hubertdeng123 hubertdeng123 force-pushed the hubertdeng123/use-transactions-python-consumer branch from e5a51ba to 5b302c8 Compare March 19, 2024 23:12
@hubertdeng123
Copy link
Member Author

nvm, stats page fix should be here:
#2909

@hubertdeng123
Copy link
Member Author

hubertdeng123 commented Mar 19, 2024

I'm going to keep this open for now and merge if folks are still experiencing issues with transactions

@zKoz210
Copy link
Contributor

zKoz210 commented Mar 21, 2024

I noticed a significant degradation in processing speed. Now events are displayed after an hour. At a measly 400-500 rps

@hubertdeng123
Copy link
Member Author

@zKoz210 is this after switching to rust python transactions consumer? Or using the python transactions python consumer?

@zKoz210
Copy link
Contributor

zKoz210 commented Mar 21, 2024

After changing to rust, python works for me without any problems.

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

Successfully merging this pull request may close these issues.

None yet

4 participants