Opened 3 years ago

Closed 7 months ago

#25091 closed enhancement (wontfix)

Make 'out/update' deterministic across instances

Reported by: iwakeh Owned by: metrics-team
Priority: Medium Milestone:
Component: Metrics/Onionoo Version:
Severity: Normal Keywords:
Cc: Actual Points: 0.1
Parent ID: #25002 Points:
Reviewer: Sponsor:

Description

Derived from comments 9 to 11 in #25002.
Summary:
Instead of current system time use the latest time of the two valid after timestamps (relays and bridges).

Child Tickets

Change History (1)

comment:1 Changed 7 months ago by karsten

Actual Points: 0.1
Resolution: wontfix
Status: newclosed

Or let's not do it. I already had a working patch, but then realized that the timestamp in out/update is used for different two purposes: 1) for server.NodeIndexer to quickly check whether there's new data to index and 2) for the various writer.*DocumentWriter to obtain a list of status files that were written or updated after the last execution. We cannot simply use the maximum last-seen time as timestamp in the out/update file, or we'd rewrite lots of document files from status files that haven't changed. We could have two separate files status/update and out/update for the two different purposes. But is that really worth the effort, just to make out/update deterministic across instances? I don't think so. Closing as wontfix.

Note: See TracTickets for help on using tickets.