[tor-bugs] #20395 [Metrics/Library]: Add capability to handle large descriptor files
Tor Bug Tracker & Wiki
blackhole at torproject.org
Tue Feb 13 16:39:13 UTC 2018
#20395: Add capability to handle large descriptor files
-----------------------------+--------------------------------
Reporter: iwakeh | Owner: karsten
Type: defect | Status: needs_revision
Priority: Medium | Milestone:
Component: Metrics/Library | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: iwakeh | Sponsor:
-----------------------------+--------------------------------
Comment (by karsten):
Hmm. I see the benefits of such an API change. And we already have #22678
that describes something very close, if not the same.
But should we separate the API change from this fix? In a way, I consider
it a bug that we're parsing an entire descriptor file before passing all
descriptors to the application. It's something we should have changed when
taking out `DescriptorFile`.
How about I rewrite my patch to not make any changes to the API (in
particular `DescriptorParser`), and then we discuss the API change,
including how to do it in a backward-compatible fashion, as part of
#22678?
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/20395#comment:19>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list