seiyak/hibernate-ogm — comment on commit 0ded1effd0c4552fcabe7b36b60626e533857875
File: hibernate-ogm-core/src/main/java/org/hibernate/ogm/persister/EntityKeyBuilder.java line 41
By seiyak:

The reason that I took this way was because of
VoldemortDatastoreProvider.getEntityMap which is called by
org.hibernate.ogm.test.utils.getEntityMap.

With my modification for EntityKey, the constructor additionally requires
columnMap to keep track of if the column name is different from the
property name.EntityKeyBuilder builds entity keys and it's
fromPersisterIdmethod receives
OgmEntityPersister as a parameter which holds the column names and entity
name. These two things are necessary to instantiate EntityKey with the
constructor. Since VoldemortDatastoreProvider.getEntityMap instantiates
EntityKey during the call, I needed to modify this way.

On Fri, Mar 2, 2012 at 7:49 AM, Emmanuel Bernard <
reply@reply.github.com
> wrote:

> What are your reasons for this hack? I am wondering if we need to provide
> some facilities but I am not sure of your intent.
>
> Ideally isolate this type of code change to a separate commit in Git so
> that we can easily surgically remove it.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/seiyak/hibernate-ogm/commit/0ded1effd0c4552fcabe7b36b60626e533857875#commitcomment-1035429
>

--
Thank you
Seiya