Emmanuel Bernard, With the changes for OGM-541 we honor now the current flush mode, i.e. if set to AUTO we do flush concerned tables before querying them, but e.g. for MANUAL we don't. So a user could set it to something other than ALWAYS or AUTO if they don't want flushes before queries.
But IIUC, you suggest that we don't actually do the flush when AUTO (or ALWAYS) is set and the store doesn't support transactions. I think that makes sense and is inline with what the spec says. I'll add a method for that to GridDialect. A simple boolean as return type would suffice for now, but I feel we should still go for an enum to allow for the addition of other possible strategies later on. Any thoughts?
|