[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #21272 [Metrics]: Onionperf deployment



#21272: Onionperf deployment
-------------------------+------------------------------
 Reporter:  hiro         |          Owner:  metrics-team
     Type:  enhancement  |         Status:  needs_review
 Priority:  Medium       |      Milestone:
Component:  Metrics      |        Version:
 Severity:  Normal       |     Resolution:
 Keywords:               |  Actual Points:
Parent ID:               |         Points:
 Reviewer:               |        Sponsor:
-------------------------+------------------------------

Comment (by karsten):

 Replying to [comment:27 iwakeh]:
 > Replying to [comment:26 karsten]:
 > > Replying to [comment:25 iwakeh]:
 > > > Couldn't `downloadFromOnionPerfHost` do some of the filename
 checking before
 > > > calling `downloadAndParseOnionPerfTpfFile`?
 > >
 > > Well, that wouldn't change functionality but would be a simple
 refactoring, right?  What's the goal there?  Make methods more testable or
 easier to read or something else?  In any case, would you want to suggest
 new methods, and I'll move around code?  Or do you want to work on a
 patch?
 >
 > No, I intended to avoid the [https://gitweb.torproject.org/karsten
 /metrics-
 db.git/tree/src/main/java/org/torproject/collector/torperf/TorperfDownloader.java?h=task-21272&id=3318eb8ca769392cca1a6ddc3344c43eba62da91#n750
 superfluous parsing] of the URL for each file and avoid download when the
 filename doesn't make sense.
 > No refactoring.

 Hmm, I'm a bit lost what you mean here.  We only download if the filename
 makes sense.  But this is discussion has become very theoretical.  Want to
 provide a patch? :D

 > > Well, if we moved that code to metrics-lib, users wouldn't be able to
 read Torperf results from anything else than the originally named .tpf
 file.  We usually avoid dependencies on file names if we can.  This case
 is a bit different, because we're archiving .tpf files, and we should be
 certain that they contain what they say.  I'd say that's specific to the
 CollecTor case though and cannot be generalized in metrics-lib.
 >
 > There could be a boolean parameter for strict checking?
 > But, I didn't mean to hijack this ticket.  I can just write that down on
 a list on my desk ;-)

 Sounds like a new ticket.  But I'm not yet sold on the idea.  If it's
 something that only CollecTor needs as provider of .tpf files and none of
 the consumers, then we shouldn't add make it part of metrics-lib.  Of
 course, if there's a plausible use case for consumers, happy to
 reconsider.  But, new (metrics-lib) ticket?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21272#comment:28>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs