[Cs5500] Tournament class updates

[Karl Lieberherr]

Hi Rick:

thank you for the updates. Yes, that parallel games feature would be
great to have.
But only if it does not jeopardize the acceptance test.

-- Karl

On Mon, Apr 11, 2011 at 9:30 PM, Rick Keilty <rkeilty at ccs.neu.edu> wrote:
> Greetings class,
>
> I've made some changes to the Tournament class and I wanted to let you know
> the reasons behind it.  Previously, when the tournament manager set a
> tournament to the running state, and called the start() method of the
> tournament, that was a blocking call and was preventing the main
> TournamentManager thread from continuing to update the status of other
> tournaments.
>
> I made the Tournament class Runnable (which we had discussed in a previous
> class), and now spawned it off into its own thread, so it isn't blocking,
> and so we know when the Tournament has completed (basically when the thread
> dies).
>
> One more change the implementation team might want to make if they have
> time, it didn't look like the Tournament class was allowing parallel games
> to be run (the BinaryGame class wasn't Runnable, and Tournament seemed to
> simply iterate over all the players and start a game between then in
> sequence.)  I know we discussed making the games parallel in their own
> threads, so if the implementation team has time this might be a nice
> enhancement.  However, we're so far along in the semester this might not be
> a worthwhile change to make, since we'll be doing some test runs on
> Wednesday evening.  I just wanted to bring it to your attention in case you
> have some extra time.
>
> Cheers,
> Rick & Greek Gods
>
> _______________________________________________
> Cs5500 mailing list
> Cs5500 at lists.ccs.neu.edu
> https://lists.ccs.neu.edu/bin/listinfo/cs5500
>
>