[metrics-bugs] #23421 [Metrics/CollecTor]: Use persistence functionality throughout all modules

[Tor Bug Tracker & Wiki]

#23421: Use persistence functionality throughout all modules
-------------------------------+-----------------------------------
 Reporter:  iwakeh             |          Owner:  metrics-team
     Type:  enhancement        |         Status:  needs_information
 Priority:  High               |      Milestone:
Component:  Metrics/CollecTor  |        Version:
 Severity:  Normal             |     Resolution:
 Keywords:  metrics-2017       |  Actual Points:
Parent ID:                     |         Points:
 Reviewer:                     |        Sponsor:
-------------------------------+-----------------------------------

Comment (by iwakeh):

 Replying to [comment:10 karsten]:
 > ...
 > How about we don't enable syncing for the new webstats module at all?
 Let's face it, it's not a use case we're planning to support, so why
 should we write or keep the necessary code to do it?

 I think CollecTor should provide syncing for all modules.  This might not
 be necessary for the operation of tp.o instances, but is a valid use case
 for maybe research or other non-TorMetrics uses.
 The sync-functionality is designed in a way that it hardly needs any code
 to add a new module, integrating a module to sync is very easy and cheap
 in terms of development time.

 >
 > And going even one step further (out of scope for this ticket), how
 about we disable syncing for all other modules ''except'' for the
 relaydescs module where we turn it into yet another data source like
 downloading from the authorities or reading from cached descriptors files?
 We could still keep the code in a form that we can add re-use it in other
 modules in the future, but only as long as that doesn't make the overall
 code more complex than it has to be. But: new ticket. Just writing this
 here to discuss the general direction.

 Again, adding sync is hardly any coding.  Making sync available for all
 descriptor types is very easy and a benefit in general.  Whether it should
 be turned on for the various tp.o  CollecTor instances is a different
 question.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/23421#comment:11>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online