JournalStoreVsJdbcStoreOnlineTest fix

[ChesterSVK]

Fix for issue: #166

Use-journal-store and use-jdbc-store attributes must be set exclusively and server must be restarted for avoiding this conflict.

[Ladicek]

Should we also update the commands? At least the builder could fail if both are journal store and JDBC store are to be enabled at the same time (which wouldn't help in this case), and maybe the command could even disable JDBC when enabling journal (and vice versa).

I'd be slightly concerned about no longer testing the original behavior (where the server would disable JDBC when enabling journal automatically), but that's probably the price of progress.

[olukas]

+1 for check in builder whether only one of journal store and JDBC store is configured. However I think that disabling JDBC when enabling journal and vice versa can cause some tricky issues for users - e.g. in case when they accidentally try to configure both store - in that case I believe that just thrown Exception is better than use last configured value for store.

@ChesterSVK As I said in #166 - ChangeJdbcAttributesOnlineTest and ChangeBasicAttributesOnlineTest intermittently fail with the same failure-description. Could you please fix also those test cases as part of this PR?

[Ladicek]

However I think that disabling JDBC when enabling journal and vice versa can cause some tricky issues for users

Yea, should probably just document that WildFly changed behavior in this.

[olukas]

@ChesterSVK Hi, is there any progress with fixing ChangeJdbcAttributesOnlineTest and ChangeBasicAttributesOnlineTest please?

[rsvoboda]

@msve please take a look

@Ladicek can you grant rights for creaper repo to @msve ?
He is "component owner" now on eap qe side.

[mnovak1]

LGTM