#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

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
tor-bugs mailing list

Reply via email to