Refactor TicketHolders to not use globals

Currently ticketholders.h is maintained as a struct with global variables. Instead it should more accurately be a decoration of ServiceContext.

There's an example of how to deal with on_update IDL interactions and global variables, so we should be able to refactor this into not having global static variables.