-
Notifications
You must be signed in to change notification settings - Fork 109
JBTM-3803 re-enable narayana-spring-boot quickstart #489
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
* @author <a href="mailto:[email protected]">Zheng Feng</a> | ||
*/ | ||
@Configuration | ||
public class DbcpXADataSourceWrapper implements XADataSourceWrapper { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleted as already injected by NarayanaAutoConfiguration by dev.snowdrop
Started testing this pull request: https://jenkins-csb-narayana-ci.dno.corp.redhat.com/job/btny-pulls-narayana-quickstart/437/ |
private TransactionManager tm; | ||
|
||
@Override | ||
public DataSource wrapDataSource(XADataSource xaDataSource) throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using GenericXADataSourceWrapper by dev.snowdrop instead (see https://github.com/snowdrop/narayana-spring-boot/blob/fe90db50f0b1f706565cc768fbf63eda5df0aa96/narayana-spring-boot-core/src/main/java/dev/snowdrop/boot/narayana/core/jdbc/GenericXADataSourceWrapper.java#L100)
de30bae
to
00f4897
Compare
Started testing this pull request: https://jenkins-csb-narayana-ci.dno.corp.redhat.com/job/btny-pulls-narayana-quickstart/438/ |
https://issues.redhat.com/browse/JBTM-3803
Using dev.snowdrop dependency for narayana-spring-boot
to re-enable: