Skip to content

get_boundary_times: add @memoize#114

Merged
macbre merged 1 commit intomasterfrom
cache-get_boundary_times
Feb 2, 2018
Merged

get_boundary_times: add @memoize#114
macbre merged 1 commit intomasterfrom
cache-get_boundary_times

Conversation

@macbre
Copy link
Owner

@macbre macbre commented Feb 2, 2018

Otherwise check_data_not_updated_recently and check_data_too_old will run these (possibly heavy) queries twice:

2018-02-02 12:39:24 indexdigest.database.query          INFO     SELECT /* index-digest */ UNIX_TIMESTAMP(MIN(`msg_date`)) as `min`, UNIX_TIMESTAMP(MAX(`msg_date`)) as `max` FROM messages_status

@macbre macbre added this to the v1.1 milestone Feb 2, 2018
@macbre macbre merged commit 5a9fd95 into master Feb 2, 2018
@macbre macbre deleted the cache-get_boundary_times branch February 2, 2018 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant