Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4)

Issue 343080043: [plaso] Added tasks profiler #1858 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 years, 12 months ago by Joachim Metz
Modified:
5 years, 11 months ago
Reviewers:
onager
CC:
kiddi, log2timeline-dev_googlegroups.com, romaing
Visibility:
Public.

Description

[plaso] Added tasks profiler #1858 Depends on: * https://codereview.appspot.com/345920043/

Patch Set 1 : Added tasks profiler #

Patch Set 2 : Added tasks profiler #

Unified diffs Side-by-side diffs Delta from patch set Stats (+155 lines, -12 lines) Patch
M plaso/cli/helpers/profiling.py View 2 chunks +2 lines, -2 lines 0 comments Download
M plaso/containers/tasks.py View 1 1 chunk +1 line, -1 line 0 comments Download
M plaso/engine/configurations.py View 1 chunk +8 lines, -0 lines 0 comments Download
M plaso/engine/profilers.py View 1 chunk +20 lines, -0 lines 0 comments Download
M plaso/multi_processing/base_process.py View 3 chunks +9 lines, -0 lines 0 comments Download
M plaso/multi_processing/task_engine.py View 1 6 chunks +34 lines, -9 lines 0 comments Download
M plaso/multi_processing/task_manager.py View 1 8 chunks +44 lines, -0 lines 0 comments Download
M plaso/multi_processing/worker_process.py View 1 2 chunks +6 lines, -0 lines 0 comments Download
M tests/engine/configurations.py View 1 chunk +5 lines, -0 lines 0 comments Download
M tests/engine/profilers.py View 2 chunks +23 lines, -0 lines 0 comments Download
M tests/multi_processing/task_manager.py View 1 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 5
Joachim Metz
5 years, 12 months ago (2018-04-30 18:27:29 UTC) #1
Joachim Metz
Code updated.
5 years, 12 months ago (2018-05-01 05:10:56 UTC) #2
Joachim Metz
Code updated.
5 years, 11 months ago (2018-05-15 04:33:09 UTC) #3
onager
LGTM
5 years, 11 months ago (2018-05-19 20:29:07 UTC) #4
Joachim Metz
5 years, 11 months ago (2018-05-20 04:04:36 UTC) #5
Changes have been merged with master branch. To close the review and clean up
the feature branch you can run: review.py close dbmerge7
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b