Skip to content
This repository has been archived by the owner on Jul 11, 2022. It is now read-only.

Bug 1312405 - EAP7 - unable to create xa datasource - At least one xa-datasource-property is required #215

Merged
merged 2 commits into from Mar 14, 2016

Conversation

tsegismont
Copy link
Member

Bug 1312405 - EAP7 - unable to create xa datasource - At least one xa-datasource-property is required

EAP7, unlike EAP6, wants xa-datasource-properties to be part of the creation operation.
Same applies to regular datasource.

…-datasource-property is required

EAP7, unlike EAP6, wants xa-datasource-properties to be part of the creation operation
import static org.rhq.modules.plugins.wildfly10.DatasourceComponent.getConnectionPropertiesAsMap;
import static org.rhq.modules.plugins.wildfly10.DatasourceComponent.isXADatasourceResource;
import static org.rhq.core.domain.resource.CreateResourceStatus.*;
import static org.rhq.modules.plugins.wildfly10.DatasourceComponent.*;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You've forgot Eclipse settings, no * imports.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le 11/03/2016 21:59, Michael Burman a écrit :

You've forgot Eclipse settings, no * imports.

Sorry I could not find the setting in IDEA's Eclipse Formatter plugin.
Do you remember where I can configure this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code style -> Java -> "Class count to use *" etc. set a high value to those.

@tsegismont
Copy link
Member Author

@burmanm IDE setup fixed and PR updated

@tsegismont
Copy link
Member Author

@burmanm Thanks for the tips!

burmanm added a commit that referenced this pull request Mar 14, 2016
Bug 1312405 - EAP7 - unable to create xa datasource - At least one xa-datasource-property is required
@burmanm burmanm merged commit e07852a into rhq-project:master Mar 14, 2016
@tsegismont tsegismont deleted the bug/1312405 branch March 14, 2016 10:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants