Cleanup the meaning of "key" vs. "id" in our entity classes

We now use "Foo.Id" of any keys which are numerical, and have
getId() accessors to read those keys.  For any other keys we
use "Foo.Key" and read them with getKey().

All PrimaryKey and SecondaryKey queries now use get() as their
access method, rather than byFoo in some cases and get() in
others.  This simplifies callers when they are doing key based
lookups for an entity.

Signed-off-by: Shawn O. Pearce <sop@google.com>
43 files changed