[tor-bugs] #22983 [Metrics/Library]: add a descriptor interface and implementation for web-logs

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Sep 19 08:54:28 UTC 2017

#22983: add a descriptor interface and implementation for web-logs
 Reporter:  iwakeh           |          Owner:  metrics-team
     Type:  enhancement      |         Status:  needs_information
 Priority:  Medium           |      Milestone:  metrics-lib 2.2.0
Component:  Metrics/Library  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:                   |        Sponsor:

Comment (by karsten):

 Replying to [comment:31 iwakeh]:
 > We agreed somewhere above to store only decompressed bytes in-memory.
 The method `public byte[] getCompressionType();` becomes obsolete in this
 case and will be removed.  Did I overlook any use for this method?

 Uhm, good question. A search on this page shows 80 matches of the word
 "compress". So, not sure if we agreed on this, but couldn't we store the
 ''compressed'' bytes in memory and ''decompress'' on-the-fly when they're
 requested? That would still not make a `getCompressionType()` mandatory,
 because the descriptor could as well store this type internally and not
 give it out. But, this is just off the top of my head, not based on re-
 reading the entire discussion above.

 However, another thought: ''adding'' a new method is a minor API change,
 whereas ''removing'' a method is a major API change. Maybe, if we can live
 without the method for now, let's try that.

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

More information about the tor-bugs mailing list