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

Re: [tor-bugs] #26162 [Metrics/CollecTor]: Replace Gson with Jackson in CollecTor

#26162: Replace Gson with Jackson in CollecTor
-------------------------------+--------------------------------
 Reporter:  karsten            |          Owner:  karsten
     Type:  enhancement        |         Status:  needs_revision
 Priority:  High               |      Milestone:
Component:  Metrics/CollecTor  |        Version:
 Severity:  Normal             |     Resolution:
 Keywords:                     |  Actual Points:
Parent ID:                     |         Points:
 Reviewer:                     |        Sponsor:
-------------------------------+--------------------------------
Changes (by iwakeh):

 * status:  needs_review => needs_revision


Comment:

 CollecTor already uses all o.tp.descriptor.index classes.  Why duplicate
 the writing functionality?  Maybe, this should be implemented by using
 only o.tp.descriptor.index classes in CollecTor?  Future json processing
 lib changes would not affect CollecTor anymore (except for simply
 including the dependency).

 Setting to needs_revision for discussion.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/26162#comment:2>
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