11

Simplified example:

I have a to-do. It can be future, current, or late based on what time it is.

  Time       State
  8:00 am    Future
  9:00 am    Current
  10:00 am   Late

So, in this example, the to-do is "current" from 9 am to 10 am.

Originally, I thought about adding fields for "current_at" and "late_at" and then using an instance method to return the state. I can query for all "current" todos with now > current and now < late.

In short, I'd calculate the state each time or use SQL to pull the set of states I need.

If I wanted to use a state machine, I'd have a set of states and would store that state name on the to-do. But, how would I trigger the transition between states at a specific time for each to-do?

  • Run a cron job every minute to pull anything in a state but past the transition time and update it
  • Use background processing to queue transition jobs at the appropriate times in the future, so in the above example I would have two jobs: "transition to current at 9 am" and "transition to late at 10 am" that would presumably have logic to guard against deleted todos and "don't mark late if done" and such.

Does anyone have experience with managing either of these options when trying to handle a lot of state transitions at specific times?

It feels like a state machine, I'm just not sure of the best way to manage all of these transitions.

Update after responses:

  • Yes, I need to query for "current" or "future" todos
  • Yes, I need to trigger notifications on state change ("your todo wasn't to-done")

Hence, my desire to more of a state-machine-like idea so that I can encapsulate the transitions.

6 Answers 6

6

I have designed and maintained several systems that manage huge numbers of these little state machines. (Some systems, up to 100K/day, some 100K/minute)

I have found that the more state you explicitly fiddle with, the more likely it is to break somewhere. Or to put it a different way, the more state you infer, the more robust the solution.

That being said, you must keep some state. But try to keep it as minimal as possible.

Additionally, keeping the state-machine logic in one place makes the system more robust and easier to maintain. That is, don't put your state machine logic in both code and the database. I prefer my logic in the code.

Preferred solution. (Simple pictures are best).

For your example I would have a very simple table:

task_id, current_at, current_duration, is_done, is_deleted, description...

and infer the state based on now in relation to current_at and current_duration. This works surprisingly well. Make sure you index/partition your table on current_at.

Handling logic on transition change

Things are different when you need to fire an event on the transition change.

Change your table to look like this:

task_id, current_at, current_duration, state, locked_by, locked_until, description...

Keep your index on current_at, and add one on state if you like. You are now mangling state, so things are a little more fragile due to concurrency or failure, so we'll have to shore it up a little bit using locked_by and locked_until for optimistic locking which I'll describe below.

I assume your program will fail in the middle of processing on occassion—even if only for a deployment.

You need a mechanism to transition a task from one state to another. To simplify the discussion, I'll concern myself with moving from FUTURE to CURRENT, but the logic is the same no matter the transition.

If your dataset is large enough, you constantly poll the database to discover to discover tasks requiring transition (of course, with linear or exponential back-off when there's nothing to do); otherwise you use or your favorite scheduler whether it is cron or ruby-based, or Quartz if you subscribe to Java/Scala/C#.

Select all entries that need to be moved from FUTURE to CURRENT and are not currently locked.

(updated:)

-- move from pending to current
select task_id
  from tasks
 where now >= current_at
   and (locked_until is null OR locked_until < now)
   and state == 'PENDING'
   and current_at >= (now - 3 days)         -- optimization
 limit :LIMIT                               -- optimization

Throw all these task_ids into your reliable queue. Or, if you must, just process them in your script.

When you start to work on an item, you must first lock it using our optimistic locking scheme:

update tasks
   set locked_by = :worker_id     -- unique identifier for host + process + thread
     , locked_until = now + 5 minutes -- however this looks in your SQL langage
 where task_id = :task_id         -- you can lock multiple tasks here if necessary
   and (locked_until is null OR locked_until < now) -- only if it's not locked!

Now, if you actually updated the record, you own the lock. You may now fire your special on-transition logic. (Applause. This is what makes you different from all the other task managers, right?)

When that is successful, update the task state, make sure you still use the optimistic locking:

update tasks
   set state = :new_state
     , locked_until = null -- explicitly release the lock (an optimization, really)
 where task_id = :task_id
   and locked_by = :worker_id -- make sure we still own the lock
                              -- no-one really cares if we overstep our time-bounds

Multi-thread/process optimization

Only do this when you have multiple threads or processes updating tasks in batch (such as in a cron job, or polling the database)! The problem is they'll each get the similar results from the database and will then contend to lock each row. This is inefficient both because it will slow down the database, and because you have threads basically doing nothing but slowing down the others.

So, add a limit to how many results the query returns and follow this algorithm:

results = database.tasks_to_move_to_current_state :limit => BATCH_SIZE
while !results.empty
    results.shuffle! # make sure we're not in lock step with another worker
    contention_count = 0
    results.each do |task_id|
        if database.lock_task :task_id => task_id
           on_transition_to_current task_id
        else
           contention_count += 1
        end
        break if contention_count > MAX_CONTENTION_COUNT # too much contention!
    done
    results = database.tasks_to_move_to_current_state :limit => BATCH_SIZE
end

Fiddle around with BATCH_SIZE and MAX_CONTENTION_COUNT until the program is super-fast.


Update:

The optimistic locking allows for multiple processors in parallel.

By have the lock timeout (via the locked_until field) it allows for failure while processing a transition. If the processor fails, another processor is able to pick up the task after a timeout (5 minutes in the above code). It is important, then, to a) only lock the task when you are about to work on it; and b) lock the task for how long it will take to do the task plus a generous leeway.

The locked_by field is mostly for debugging purposes, (which process/machine was this on?) It is enough to have the locked_until field if your database driver returns the number of rows updated, but only if you update one row at a time.

4
  • Preferred solution: how I'm doing it now. I go one step further and denormalize current_at to "start_at, current_at, late_at" and so on. But, we're outgrowing running them all in one script (good problem to have!) and I'm looking for solutions to decouple things. I love "the more state you infer, the more robust the solution." Very good advice! Commented Nov 15, 2011 at 1:32
  • What kind of things make the script outgrown? Events on transition from one state to another? Commented Nov 15, 2011 at 2:48
  • @wesgarrison the solution I outlined was designed to handle high concurrency situations so you could easily add another thread/process. Thinking more about your use case, I would just process things in the script and scrap the reliable queue—unless you have multiple events per state transition. Even then, you could add more states instead of the queue, but that may mess up some abstractions Commented Nov 15, 2011 at 17:25
  • I'm sending emails on transitions and they take longer/sometimes fail, plus if something goes awry with a single transition it can kill the whole script and nothing after it gets processed, unless I begin/rescue the whole thing, then pass that error state off, etc etc etc. Instead, I figured I would try to think of a better way to do it. Commented Nov 16, 2011 at 5:55
4

Managing all those transitions at specific times does seem tricky. Perhaps you could use something like DelayedJob to schedule the transitions, so that a cron job every minute wouldn't be necessary, and recovering from a failure would be more automated?

Otherwise - if this is Ruby, is using Enumerable an option?

Like so (in untested pseudo-code, with simplistic methods)

ToDo class

def state
  if to_do.future?
    return "Future"
  elsif to_do.current?
    return "Current"
  elsif to_do.late?
    return "Late"
  else 
    return "must not have been important"
  end
end

def future?
    Time.now.hour <= 8
end

def current?
    Time.now.hour == 9
end

def late?
    Time.now.hour >= 10
end

def self.find_current_to_dos
    self.find(:all, :conditions => " 1=1 /* or whatever */ ").select(&:state == 'Current')
end
4
  • The issue with this is I'd be selecting ALL the items (which instantiates them and uses memory) just to whittle down the set with select(). I've definitely considered the DelayedJob approach, but that's a LOT of little jobs running all the time, spinning up just to do one thing. Commented Nov 15, 2011 at 1:25
  • Agreed, lots of little jobs. And your point about instantiating the objects is well taken. One great thing about DelayedJob is that it only spins up the Rails environment once (per worker), so its overhead/lag is small. Commented Nov 15, 2011 at 17:55
  • Also, if I had three transition jobs in my db for every todo I have now, I'd be at 1.5M rows in that table. Commented Nov 16, 2011 at 5:59
  • This is a nice, simple solution if you aren't dealing with huge amounts of data. Commented Feb 16, 2015 at 19:15
3
+50

One simple solution for moderately large datasets is to use a SQL database. Each todo record should have a "state_id", "current_at", and "late_at" fields. You can probably omit the "future_at" unless you really have four states.

This allows three states:

  1. Future: when now < current_at
  2. Current: when current_at <= now < late_at
  3. Late: when late_at <= now

Storing the state as state_id (optionally make a foreign key to a lookup table named "states" where 1: Future, 2: Current, 3: Late) is basically storing de-normalized data, which lets you avoid recalculating the state as it rarely changes.

If you aren't actually querying todo records according to state (eg ... WHERE state_id = 1) or triggering some side-effect (eg sending an email) when the state changes, perhaps you don't need to manage state. If you're just showing the user a todo list and indicating which ones are late, the cheapest implementation might even be to calculate it client side. For the purpose of answering, I'll assume you need to manage the state.

You have a few options for updating state_id. I'll assume you are enforcing the constraint current_at < late_at.

  • The simplest is to update every record: UPDATE todos SET state_id = CASE WHEN late_at <= NOW() THEN 3 WHEN current_at <= NOW() THEN 2 ELSE 1 END;.

  • You probably will get better performance with something like (in one transaction) UPDATE todos SET state_id = 3 WHERE state_id <> 3 AND late_at <= NOW(), UPDATE todos SET state_id = 2 WHERE state_id <> 2 AND NOW() < late_at AND current_at <= NOW(), UPDATE todos SET state_id = 1 WHERE state_id <> 1 AND NOW() < current_at. This avoids retrieving rows that don't need to be updated but you'll want indices on "late_at" and "future_at" (you can try indexing "state_id", see note below). You can run these three updates as frequently as you need.

  • Slight variation of the above is to get the IDs of records first, so you can do something with the todos that have changed states. This looks something like SELECT id FROM todos WHERE state_id <> 3 AND late_at <= NOW() FOR UPDATE. You should then do the update like UPDATE todos SET state_id = 3 WHERE id IN (:ids). Now you've still got the IDs to do something with later (eg email a notification "20 tasks have become overdue").

  • Scheduling or queuing update jobs for each todo (eg update this one to "current" at 10AM and "late" at 11PM) will result in a lot of scheduled jobs, at least two times the number of todos, and poor performance -- each scheduled job is updating only a single record.

  • You could schedule batch updates like UPDATE state_id = 2 WHERE ID IN (1,2,3,4,5,...) where you've pre-calculated the list of todo IDs that will become current near some specific time. This probably won't work out so nicely in practice for several reasons. One being some todo's current_at and late_at fields might change after you've scheduled updates.

Note: you might not gain much by indexing "state_id" as it only divides your dataset into three sets. This is probably not good enough for a query planner to consider using it in a query like SELECT * FROM todos WHERE state_id = 1.

The key to this problem that you didn't discuss is what happens to completed todos? If you leave them in this todos table, the table will grow indefinitely and your performance will degrade over time. The solution is partitioning the data into two separate tables (like "completed_todos" and "pending_todos"). You can then use UNION to concatenate both tables when you actually need to.

1
  • Some clarifications above, but some specific comments as well. In practice, yes, we need the history of todos, but they're scoped by other fields that will be indexed so we won't be hitting all [currently] half-million records each time. Commented Nov 15, 2011 at 1:23
2

State machines are driven by something. user interaction or the last input from a stream, right? In this case, time drives the state machine. I think a cron job is the right play. it would be the clock driving the machine.

for what it's worth it is pretty difficult to set up an efficient index on a two columns where you have to do a range like that.

now > current && now < late is going to be hard to represent in the database in a performant way as an attribute of task

id|title|future_time|current_time|late_time

1|hello|8:00am|9:00am|10:00am

1

Never try to force patterns into problems. Things are the other way around. So, go directly to find a good solution for it.

Here is an idea: (for what I understood yours is)

Use persistent alerts and one monitored process to "consume" them. Secondarily, query them.

That will allow you to:

  1. keep it simple
  2. keep it cheap to maintain. Secondarily it also will keep you mentally more fresh to do something else.
  3. keep all the logic in code only (as it should).

I stress the point of having that process monitored with some kind of watchdog so you are ensured to send those alerts in time (or, in a worst case scenario, with some delay after a crash or things like that).

Note that: the fact of having persisted those alerts allows you this two things:

  1. make/keeps your system resilient (more fault tolerant) and
  2. make you able to query future and current items (by playing around with querying the alerts' time range as best fits your needs)
2
  • I like this idea a lot, but I still struggle with 500,000 todos (so far) times at least 3 transitions = lots of records in my alerts table. Maybe I don't have to worry about that for now, but it's another thing to test out before I implement it. Thanks for the suggestion! Commented Nov 16, 2011 at 5:58
  • Sure :) I think is one of those things that you can't really know until you try Commented Nov 23, 2011 at 16:30
0

In my experience, a state machine in SQL is most useful when you have an external process acting on something, and updating the database with it's state. For example, we have a process that uploads and converts videos. We use the database to keep track of what is happening to a video at any time, and what should happen to it next.

In your case, I think you can (and should) use SQL to solve your problem instead of worrying about using a state machine:

Make a todo_states table:

todo_id  todo_state_id    datetime  notified
1        1 (future)       8:00      0
1        2 (current)      9:00      0
1        3 (late)         10:00     0

Your SQL query, where all the real work happens:

SELECT todo_id, MAX(todo_state_id) AS todo_state_id 
FROM todo_states
WHERE time < NOW()
GROUP BY todo_id

The currently active state is always the one you select. If you want to notify the user just once, insert the original state with notify = 0, and bump it on the first select.

Once the task is "done", you can either insert another state into the todo_states table, or simply delete all the states associated with a task and raise a "done" flag in the todo item, or whatever is most useful in your case.

Don't forget to clean out stale states.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.