The IDGenerator currently returns an Object, when it would be better practice to make this generic:

public interface IdGenerator<T> {
T generate();
}

There are places where we need to resort to using Objects, but this is not one of them.