#51194 Issue 51165 - add more logconv stats for the new access log keywords
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base 51165  into  master

Download 51194.patch

Description: Add "average" stats for wtime, optime, and etime

relates: https://pagure.io/389-ds-base/issue/51165

Reviewed by: ?

Ack from me.

As an aside, at what point do we rewrite logconv to py? Sorry to open that bag of fun :)

Ack from me.
As an aside, at what point do we rewrite logconv to py? Sorry to open that bag of fun :)

There is a ticket open for it I believe. The issue is that logconv uses a special perl database to store it hashes, I don't think there is a python equivalent. Needs more investigation, but I don't think it will be an easy port to python (where it's fast and does not consume a ton of memory), fun fun :-)

Ahhhh cool, if we have a ticket for it, then no issue. Feel free to merge :) thanks mate,

rebased onto f771ff7e6ea97641d1abfc5816e9631b699454b3

Pull-Request has been merged by mreynolds

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/4247

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata