-
Notifications
You must be signed in to change notification settings - Fork 120
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
Laod test suite #952
Laod test suite #952
Conversation
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.
report name has date which means it is not getting overwritten?
see other comments.
.github/workflows/load-test.yml
Outdated
@@ -1,6 +1,7 @@ | |||
name: load-test | |||
|
|||
on: | |||
push: |
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.
why on push?
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.
removed, used for testing
@@ -16,6 +16,7 @@ | |||
|
|||
|
|||
public class Canopy<R> implements Serializable { | |||
private static final long serialVersionUID = -229533781044789499L; |
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.
why are we adding it here? and this value in any case is not making sense.
perf_test_input.py
Outdated
@@ -7,13 +7,14 @@ | |||
#load file config to test on | |||
config_65 = "examples/febrl120k/config.json" |
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.
rename to febrl, febrl120k, ncVoter5m
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.
done
changes
1- added time to report
2- report will be created under ./perfTestReport folder
3- every report will overwrite previous report