|
|
|
|
|
|
* There are some methods marked as experimental. Are they proven ok by now?
* {{executeBackendQuery()}} expects an array of entity key metadata. Is this correct for projections? (/)
* Should we provide abstract base classes to derive from so to allow for the addition of methods where a reasonable default behavior exists in a compatible manner (not sure whether there are such cases)?
* Should we generally hide types from ORM in OGM contracts? Specifically, should we pass query parameters to {{executeBackendQuery()}} via something else than {{org.hibernate.engine.spi.QueryParameters}}? I think we should as it exposes some SQL specifics not matching for our case.
* Is {{GridDialect#createTupleAssociation()}} really needed? It is implemented with a simple {{return new Tuple()}} in all but one cases (Neo4j). For Neo4j we could move the concerned logic to {{createAssociation()}}/{{updateAssociation()}} so that we could get rid of this method.
* Should {{GridDialect}} require a constructor with the datastore provider type or should there be a more generic solution (see discussion in {{GridDialectInitiator}}).
|
|
|
|
|
|
