Skip to content

Remove execution of Matomo middleware#660

Merged
sfisher merged 4 commits into
developfrom
remove-matomo-middleware2
Jul 2, 2024
Merged

Remove execution of Matomo middleware#660
sfisher merged 4 commits into
developfrom
remove-matomo-middleware2

Conversation

@sfisher

@sfisher sfisher commented Jun 27, 2024

Copy link
Copy Markdown
Contributor

This hotfix removes matomo middleware. IDK if we want to accept this for the time being until we implement optimizations for Matomo + N2T interactions.

This just comments it out of middleware so it is no longer loaded or active in the application.

Possible optimization instead.

  • I believe Dave suggested we do not count any of the problematic types of N2T resolutions.
  • Use a more performant data store than mySQL. I'm surprised it performed so badly under higher load, but it supports these brokers and stores which would likely perform better than than mySQL. There are additional ones, also.

https://docs.celeryq.dev/en/stable/getting-started/backends-and-brokers/index.html

Name Status Monitoring Remote control
RabbitMQ Stable Yes Yes
Redis Stable Yes Yes
Amazon SQS Stable No No
Zookeeper Experimental No No

@sfisher sfisher changed the base branch from main to develop June 27, 2024 22:04
@sfisher sfisher merged commit cf59a2f into develop Jul 2, 2024
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