Problem/Motivation

The queue functionality should work for ANY queue, not just Redis. Lets refactor this to just be queue agnostic

Issue fork usage_data-3261184

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

b_sharpe created an issue. See original summary.

levibs made their first commit to this issue’s fork.

b_sharpe’s picture

Status: Active » Needs review
b_sharpe’s picture

Status: Needs review » Needs work

I have some concerns with how this works. The queue system is meant to work without knowledge of it's backend, we should be following this as well.

What ideally we'd see here is just a selection of "Use Queue API to process usage data" as an option per UsageType plugin, in which when checked items are put into the queue and then are processed by our worker without knowledge of which backend is doing the queuing.

Queue workers will always be Drupal, but the queue backend should be up to the developer in control of such settings. Obviously our preference is not database here, as the whole point is to prevent DB interaction on every page tracking usage, which is why Redis was originally targeted.

joaoimagex made their first commit to this issue’s fork.

  • b_sharpe committed 93de81b on 2.0.x authored by joaoimagex
    Issue #3261184 by levibs, joaoimagex, b_sharpe: Refactor Queue logic to...
b_sharpe’s picture

Status: Needs work » Fixed

Awesome, Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.