-
Notifications
You must be signed in to change notification settings - Fork 558
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
Introduce List storage for engine tests #4241
Conversation
@deepthidevaki part of this PR I refactored a bit the |
Can we remove entirely dependency on Atomix from the engine forever? Next is removing it from LogStream 😍 |
@npepinpe I tried but I needed the ZeebeEntry stuff but I think it is a step forward we can iterate/improve on that |
25536f5
to
3856509
Compare
Looks good. Thanks 👍 |
@npepinpe nevermind was able to remove the atomix dep ✌️ |
@saig0 pingPong 🏓 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice improvement 👍 Thanks!
bors r+ |
4241: Introduce List storage for engine tests r=Zelldon a=Zelldon ## Description Replaces the AtomixLogStorage with an on heap ListStorage. Most of the tests are by factor 2 faster then before. For example variable.mappings tests before: ![old](https://user-images.githubusercontent.com/2758593/78557599-5db2d680-7811-11ea-9223-47d1de6e66f5.png) now: ![improve](https://user-images.githubusercontent.com/2758593/78557612-61465d80-7811-11ea-869e-1c314ff6bec3.png) The engine test suite now runs on jenkins with around 1:50 min, before 02:39 min. ## Related issues closes #4240 closes #4242 # Co-authored-by: Christopher Zell <zelldon91@googlemail.com>
Build failed |
bors r+ |
Build succeeded |
Description
Replaces the AtomixLogStorage with an on heap ListStorage.
Most of the tests are by factor 2 faster then before.
For example variable.mappings tests before:
now:
The engine test suite now runs on jenkins with around 1:50 min, before 02:39 min.
Related issues
closes #4240
closes #4242
Pull Request Checklist
mvn clean install -DskipTests
locally before committing