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

Performance improvements #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ballesterosdm
Copy link
Member

Using OpenDDR in a real environment we have detected a set of performance requirements:
1 - High-load environments need a high number of device identifications per second.
2 - In real browsing situations, the same User-Agent string is used repeatedly.

To improve the performance of OpenDDR in these situations we have worked in the addition of Cache mechanisms and the optimization in the use of Regular Expressions.

The reason that encourage the development of these improvents was the study of the logs stored in a proxy, deployed in high-load enviroments, which provide mobile web pages.

More info in the CTIC_README file.

@gounthar
Copy link

I didn't manage to get your code working well on my machine, but I would be interested in seeing the improvement.
I can identify 90 devices per second with the original code from OpenDDR (with all properties), it would be nice if it could climb to 500.

@ballesterosdm
Copy link
Member Author

Hi gounthar.

Let me know what's going wrong and I will try to help you.

@gounthar
Copy link

I managed to get it working today. It went from 90 device identifications per second to 197.
Good job!

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