codefordenver / codefordenver/Comrad
RRule performance issues
- Dominant language
- JavaScript
- Stars
- 25
- Forks
- 9
- PR merge metrics
- No merged PRs in 30d
Description
## Description of Issue
We have been using a modified version of the `rrule` library that has some performance improvements (https://github.com/seankwilliams/rrule), but it is still doesn't perform great. In an endpoint like the following:
http://localhost:5000/v1/events/traffic?startDate=%222020-02-27T00:00:00-06:00%22&endDate=%222020-02-28T00:00:00-06:00%22&filterByTrafficType%5B%5D=Announcement&filterByTrafficType%5B%5D=Feature&filterByTrafficType%5B%5D=Giveaway&filterByTrafficType%5B%5D=Legal+ID&filterByTrafficType%5B%5D=PSA&filterByTrafficType%5B%5D=Underwriting
Most of the response time is spent by about 1 second iterating through rule.between for each of the repeat rules being passed in.
We need to evaluate either using a different method for repeat rules or optimizing the existing library.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.