Opened 3 years ago

Closed 3 years ago

#26169 closed enhancement (fixed)

Replace Gson with Jackson in Tor Metrics

Reported by: karsten Owned by: karsten
Priority: High Milestone: metrics-web 1.1.0
Component: Metrics/Website Version:
Severity: Normal Keywords:
Cc: metrics-team Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Related to Onionoo ticket #25848 and metrics-lib ticket #26159, we should replace Gson with Jackson in Tor Metrics. For reasons, see those tickets. I'll work on this.

Child Tickets

Change History (5)

comment:1 Changed 3 years ago by karsten

Status: assignedneeds_review

comment:2 Changed 3 years ago by karsten

Bumping this ticket, so that we can hopefully put out a release this week. Hint: the patch only looks huge because of the updated news.json, but there's no need to review that part except for maybe re-generating and comparing results. (Thanks!)

comment:3 Changed 3 years ago by iwakeh

Status: needs_reviewmerge_ready

Was there a trac-hickup on Friday?

I both set reviewer to 'iwakeh' and reviewed this (there is the checkout on my hard drive), but indeed never received the mailings from trac.

All was fine and is merge ready. Good to also have less empty space in news.json.

comment:4 in reply to:  3 Changed 3 years ago by karsten

Replying to iwakeh:

Was there a trac-hickup on Friday?

Plausible. I didn't see anything.

I both set reviewer to 'iwakeh' and reviewed this (there is the checkout on my hard drive), but indeed never received the mailings from trac.

All was fine and is merge ready. Good to also have less empty space in news.json.

Great! Created the release ticket and will close this one once the release is out. Thanks!

comment:5 Changed 3 years ago by karsten

Milestone: metrics-web 1.1.0
Resolution: fixed
Status: merge_readyclosed

Merged, released, and deployed. Closing.

Note: See TracTickets for help on using tickets.