[tor-bugs] #17755 [CollecTor]: Add @type annotation to each Torperf measurement
Tor Bug Tracker & Wiki
blackhole at torproject.org
Sat Dec 5 09:34:31 UTC 2015
#17755: Add @type annotation to each Torperf measurement
---------------------------+-----------------
Reporter: karsten | Owner:
Type: defect | Status: new
Priority: Medium | Milestone:
Component: CollecTor | Version:
Severity: Normal | Keywords:
Actual Points: | Parent ID:
Points: | Sponsor:
---------------------------+-----------------
According to the format description on the CollecTor website: "A Torperf
results file contains a single line per Torperf run with key=value pairs.
Such a result line is sufficient to learn about 1) the Tor and Torperf
configuration, 2) measurement results, and 3) additional information that
might help explain the results."
The Torperf Measurement Result files that CollecTor provides all start
with a `@type torperf 1.0` annotation, followed by one or more measurement
lines. However, the correct way to do it is to include a `@type torperf
1.0` annotation for each contained measurement.
The main reason is that each measurement is a descriptor in the sense that
it describes what Torperf recorded for a single measurement. The fact
that two or more such measurements end up in the same file is mostly
random and only happens because it makes measurement results easier to
handle. But the measurements that are contained in a file are not
necessarily related, as it would be okay to just throw in all measurements
from a given day or even month into a single file. The line is the
descriptor, not the file, and that's why each line deserves its own
`@type` annotation. This would be more obvious if measurements were to
span multiple lines, but that doesn't mean that we can optimize the
annotation away in the single-line case.
A possible reason against making this change is file size. I tested
adding these annotations to the November 2015 tarball. The uncompressed
tarball grew from 25904640 to 26491904 bytes (+2.3%), the compressed
tarball grew from 2629336 to 2641308 bytes (+0.5%). This seems not crazy.
I'll post a simple branch that adds more @type annotations later today,
and I'll replace existing tarballs with files that contain one such
annotation per included line.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/17755>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list