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

dkpro-jwpl-hibernate-5.2.1-spring-4.3.0-migration #137

Merged
merged 4 commits into from
Jul 21, 2016
Merged

dkpro-jwpl-hibernate-5.2.1-spring-4.3.0-migration #137

merged 4 commits into from
Jul 21, 2016

Conversation

mawiesne
Copy link
Contributor

@mawiesne mawiesne commented Jul 15, 2016

provides changes that upgrade JWPL to use modern framework dependencies:

  • Hibernate 5.2.1.Final (it was: 4.1.3.Final)
  • Spring 4.3.1.RELEASE (it was: 3.1.1.RELEASE)
  • applies necessary fixes that would break the code in WikiHibernateUtil
  • removes deprecated warnings in several Hibernate bound classes related to Hibernate 4.x -> 5.x update
  • updates some JDBC driver dependencies such as MySQL and PostgreSQL to more recent versions

- Hibernate 5.2.1.Final (it was: 4.1.3.Final)
- Spring 4.3.0.RELEASE (it was: 3.1.1.RELEASE)
applies necessary fixes that would break the code in WikiHibernateUtil
removes deprecated warnings in several Hibernate bound classes related to Hibernate 4.x -> 5.x update
updates some JDBC driver dependencies such as MySQL and PostgreSQL to more recent versions
@mawiesne
Copy link
Contributor Author

mawiesne commented Jul 15, 2016

Maybe it's worth releasing a '1.2.0' version to Maven central - in case the PR is acceptable for you.

Don't hesitate to comment/ask me in case of any questions on my intention behind the proposed changes.

@rzo1
Copy link
Contributor

rzo1 commented Jul 15, 2016

+1 for this pull request!

mwiesner added 2 commits July 15, 2016 12:49
… want to use that at runtime and can't inject this config then...

- adds a log4j entry to ignore warnings in JWPL as now C3P0 dependency is available at runtime
@@ -1,6 +1,6 @@
# Copyright 2016
# Ubiquitous Knowledge Processing (UKP) Lab
# Technische Universit�t Darmstadt
# Technische Universit�t Darmstadt
Copy link
Member

Choose a reason for hiding this comment

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

was that intentionally?

Copy link
Contributor Author

@mawiesne mawiesne Jul 21, 2016

Choose a reason for hiding this comment

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

no, seems the IDE screwed that one up. I'll fix that.

Copy link
Member

Choose a reason for hiding this comment

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

Java properties files must be ISO 8859. Possibly the encoding in the repository was UTF-8 and the IDE tried to "fix" that.

@daxenberger
Copy link
Member

Thanks a lot for the PR. We appreciate your interest and contributions very much.

GitHub is a great place for you to build your open source contributor profile and we love to take new contributors on board. We have the policy of asking for a contributor license agreement. As your proposed changes are substantial and maybe you plan to contribute even more, I would ask you to please could carefully review our contribution policy and contributor license agreement:

http://dkpro.github.io/contributing/

If you are able to send me a signed copy, I'll be more than happy to include your contributions into the codebase!

fixes a mis-encoded umlaut character
@mawiesne
Copy link
Contributor Author

@daxenberger - Just sent you a signed copy.

@daxenberger
Copy link
Member

Got it; thanks!

@daxenberger daxenberger merged commit 0c2604a into dkpro:master Jul 21, 2016
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.

4 participants