Skip to content

execution time stats#85

Merged
malaskowski merged 4 commits intomasterfrom
feature/execution-time-stats
May 4, 2017
Merged

execution time stats#85
malaskowski merged 4 commits intomasterfrom
feature/execution-time-stats

Conversation

@malaskowski
Copy link
Copy Markdown
Contributor

Collecting some execution time stats and present them on the report. Later it may be used to analyse time consumed for processing the suite.


I hereby agree to the terms of the AET Contributor License Agreement.

Copy link
Copy Markdown
Contributor

@subiron subiron left a comment

Choose a reason for hiding this comment

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

can we also measure time on TEST, and URL level?
I would be helpful to detect bottlenecks without doing any computation (like sum of steps duration)


import java.io.Serializable;

public class Statistics implements Serializable {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there anything else that we can measure inside Statistics class in the future?
I'm asking because it looks like we have here the whole class to wrap single field with no additional reasons.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, my idea was to add more data here in the future, e.g. start/end time if required, maybe some wait/sleep time required to pass.
I'd like to have statistics of how long AET really needed to perform a test (how long loading/sleep took vs real test time).

@malaskowski
Copy link
Copy Markdown
Contributor Author

@subiron measuring full statistics at Test and Url level is very trick and I didn't include it in this pull request.
The problem is that test and url processing (I mean both collect and compare phases) is performed in dispersed environment. AET uses (especially during comparison phase) idea similar to divide and conquer. This mean totally separate workers/nodes execute parts of test and url processing. Because of that storing Statistics for the url or test may even not make a sens - it would be better to sum up all statistics of descendant data nodes. But this can be already done with the data that is already stored in those nodes (for url, these nodes are steps, for test it would be the sum of all steps of all urls).

@malaskowski malaskowski merged commit a6c5dc5 into master May 4, 2017
@malaskowski malaskowski deleted the feature/execution-time-stats branch May 4, 2017 15:04
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