Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Significant performance improvements
Fixes an issue that was taking place for the default domain. We had an issue where we weren't allowed to add the default domain to the domain table and would cache miss on each and every redirect links.find call against the default domain.
With the standard Kutt processes, a single redirect request takes between 60 and 80 ms on a local machine, and 120 -150 ms on a small Ubuntu server.
After implementing a cache check on the default domain, a local redirect request dropped to 5-10ms.
I also found that be default there weren't indexes to the links.address column, even though that is where the query primarily takes place. Adding an index to this a local redirect request (without the cache optimizations in place) took between 7-15 ms for a read.