WordPress / openverse

Openverse is a search engine for openly-licensed media. This monorepo includes all application code.

Home Page:https://openverse.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Remove popularity & matview timeouts from data refresh configurations

AetherUnbound opened this issue · comments

Description

The DataRefresh dataclass has slots for the following fields that are vestigial and no longer necessary:

  • refresh_metrics_timeout
  • refresh_matview_timeout
  • create_pop_constants_view_timeout
  • create_materialized_view_timeout

These can all be removed from data_refresh_types.py, since they are not used:

refresh_metrics_timeout: timedelta expressing amount of time the
refresh popularity metrics tasks may take.
refresh_matview_timeout: timedelta expressing amount of time the
refresh of the popularity matview may take.
create_pop_constants_view_timeout: timedelta expressing amount of time the
creation of the popularity constants view
may take
create_materialized_view_timeout: timedelta expressing amount of time the
creation of the matview may take

refresh_metrics_timeout: timedelta = timedelta(hours=1)
refresh_matview_timeout: timedelta = timedelta(hours=1)
create_pop_constants_view_timeout: timedelta = timedelta(hours=1)
create_materialized_view_timeout: timedelta = timedelta(hours=1)

refresh_metrics_timeout=timedelta(hours=24),
refresh_matview_timeout=timedelta(hours=72),
create_pop_constants_view_timeout=timedelta(hours=24),
create_materialized_view_timeout=timedelta(hours=72),

Additional context

This was left over from #433