MongoDB + Skip by dialect

[Sanne]

This can be merged into master to include MongoDB in it.

If you don't like the idea yet, please merge it in mongodb

After merging we'll have to let know people using it that the default build does not build (nor test) the mongodb dialect, but the Maven profile mongodb needs to be activated, for example:

mvn clean install -Pmongodb

[Sanne]

@emmanuelbernard I've added a commit to resolve for https://hibernate.onjira.com/browse/OGM-166

I couldn't find any way to "hardcode" the different hostname in your profile: the project pom always has precedence.
Of course environment and parameters have precedence on the project pom, so it should work fine for you by using

build clean install -Dmongodb.hostname=yourvm

This seems to work fine for Eclipse as well, and I guess IDEA too as it's the same pattern we use in Search to inject the H2 connection properties.

P.S. you'll have to set the environment property in the IDE as well

[emmanuelbernard]

:(
Thinking again at the release plugin, I am not sure -D will work as the release plugin does fork to run some of the steps.
Do we have to put these default propertiles in the pom itself? The default values are already in our codebase. And if these are not set in the pom, maybe the setting.xml could be used.

[Sanne]

After a long fight with Maven about properties I'm tempted to propose this new approach, imho cleaner:

• no need to change your settings.xml
• define an environment property: MONGODB_PORT
• and MONGODB_HOSTNAME

The mongo module will kick in just by having these env settings in your environment. Make sure you IDE has the same environment to be able to launch unit tests.

I had to read the env directly from our code as with Maven there was no way to have both the build and the IDE working; I'm copying them to system properties and let the Hibernate configuration read them from there.
(We can't have them read env properties directly as it's illegal to define environment properties with dots)

See 2a9a9ea , which replaces the previous commit in the pull.

[emmanuelbernard]

I am closing this pull request as it is superseded by #77