Skip to content
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

Added a sample that illustrates using multiple DataSources #415

Closed
wants to merge 1 commit into from

Conversation

mminella
Copy link
Contributor

This PR adds a sample that illustrates the use of multiple DataSources
in a single Spring Cloud Task application.

Resolves #413

This PR adds a sample that illustrates the use of multiple DataSources
in a single Spring Cloud Task application.

Resolves spring-cloud#413
@mminella mminella added the in pr label Apr 13, 2018
<parent>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-parent</artifactId>
<version>2.0.0.RELEASE</version>
Copy link
Collaborator

@cppwfs cppwfs Apr 13, 2018

Choose a reason for hiding this comment

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

We can use 2.0.1.RELEASE.

@@ -0,0 +1,31 @@
/*
* Copyright 2017 the original author or authors.
Copy link
Collaborator

Choose a reason for hiding this comment

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

2018... Can't believe it myself ;-)

import org.springframework.boot.SpringApplication;
import org.springframework.boot.autoconfigure.SpringBootApplication;
import org.springframework.cloud.task.configuration.EnableTask;

Copy link
Collaborator

Choose a reason for hiding this comment

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

add author


== Requirements:

* Java 7 or Above
Copy link
Collaborator

Choose a reason for hiding this comment

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

Java 8 or above.

@cppwfs
Copy link
Collaborator

cppwfs commented Apr 13, 2018

LGTM . Will fix comments above on merge.

@cppwfs
Copy link
Collaborator

cppwfs commented Apr 13, 2018

Polished, Merged, Squashed.

@cppwfs cppwfs closed this Apr 13, 2018
@cppwfs cppwfs removed the in pr label Apr 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants