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

TEIIDSB-211 sample, addition to util and mustache #276

Closed

Conversation

aditya300899
Copy link
Member

No description provided.

@aditya300899
Copy link
Member Author

[WARNING] Found duplicate and different resources in [com.amazonaws:aws-java-sdk-swf-libraries:1.11.22, org.springframework.data:spring-data-jpa:2.2.6.RELEASE]:
[WARNING] builddef.lst
[WARNING] Found duplicate classes/resources in compile classpath.
[WARNING] Found duplicate and different resources in [com.amazonaws:aws-java-sdk-swf-libraries:1.11.22, org.springframework.data:spring-data-jpa:2.2.6.RELEASE]:
[WARNING] builddef.lst
[WARNING] Found duplicate classes/resources in runtime classpath.

I removed the dependency on spring-data-jpa because the build was failing on the 1st commit due to the above. I don't see why yet.

@aditya300899
Copy link
Member Author

I excluded the com.amazonaws:aws-java-sdk-swf-libraries:1.11.22 from aws-sdk-java considering that we won't need it as amazon simple work flow is used to easy build some cloud applications. In my opinion, this is a better workaround. What do you think?

@shawkins
Copy link
Contributor

shawkins commented Jul 6, 2020

In my opinion, this is a better workaround. What do you think?

Once we incorporate the changes from the Teiid side we should have more targeted dependencies.

@shawkins
Copy link
Contributor

shawkins commented Jul 7, 2020

Incorporated these commits into #278

@shawkins shawkins closed this Jul 7, 2020
@aditya300899 aditya300899 deleted the TEIIDSB-211-S3-Example branch July 8, 2020 06:12
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.

2 participants