Ticket #4963 (closed task: invalid)

Opened 2 years ago

Last modified 2 years ago

Faster implementation of System.Timeout.timeout

Reported by: basvandijk Owned by:
Priority: normal Milestone:
Component: libraries/base Version: 7.0.1
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Difficulty:
Test Case: Blocked By:
Blocking: Related Tickets:

Description

As described in the  thread on the libraries list I wrote a faster implementation for System.Timeout.timeout.

Attached are a patch for base and a tiny Criterion benchmark that compares the current timeout with the new one.

The patch does need some careful review!

Attachments

bench_timeouts.tar.gz Download (28.5 KB) - added by basvandijk 2 years ago.
Criterion benchmark for comparing the current with the new timeout
faster_timeout.dpatch Download (76.1 KB) - added by basvandijk 2 years ago.

Change History

Changed 2 years ago by basvandijk

Criterion benchmark for comparing the current with the new timeout

Changed 2 years ago by basvandijk

Changed 2 years ago by basvandijk

  • status changed from new to closed
  • resolution set to invalid

Since this event manager based implementation is  broken and since my earlier implementation was not actually faster on the more representative busyWontTimeout benchmark, I conclude that I can't improve on the current timeout. So I'm closing the ticket.

Note: See TracTickets for help on using tickets.