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 patch for HHH-1237 #738

Closed
wants to merge 291 commits into from
Closed

Added patch for HHH-1237 #738

wants to merge 291 commits into from

Conversation

michael-schnell
Copy link
Contributor

A patch for bug "Escaping : with :: in queries" (HHH-1237) was provided three years ago, but never applied to the code: https://hibernate.atlassian.net/browse/HHH-1237

Wasn't able to test it because I don't use Gradle (Maven wouldn't have been a problem...)

lukasz-antoniak and others added 30 commits January 24, 2014 22:38
Configuration#buildSessionFactory() in documentation
…rm) XSD;

HHH-7556 - Clean up packages;
HHH-7164 - Finish out SessionFactoryBuilder api
with
missing descriptor files. Mojo will automatically hook into build
lifecycle compile phase.

Conflicts:
	hibernate-enhance-maven-plugin/src/main/java/org/hibernate/bytecode/enhance/plugins/MavenEnhancePlugin.java
	settings.gradle
	tooling/hibernate-maven-plugin/hibernate-maven-plugin.gradle
Conflicts:
	.gitignore
	hibernate-enhance-maven-plugin/src/main/resources/META-INF/maven/plugin.xml
	hibernate-enhance-maven-plugin/src/main/resources/META-INF/maven/plugin.xml.original
	tooling/hibernate-enhance-maven-plugin/src/main/resources/META-INF/maven/plugin.xml
gbadner and others added 21 commits April 21, 2014 15:50
of SqlExceptionHelper#handleAndClearWarnings
Conflicts:
	hibernate-core/src/main/java/org/hibernate/engine/jdbc/spi/SqlExceptionHelper.java
A patch for bug "Escaping : with :: in queries" (HHH-1237) was provided three years ago, but never applied to the code: https://hibernate.atlassian.net/browse/HHH-1237
Wasn't able to test it because I don't use Gradle (Maven wouldn't have been a problem...)
@brmeyer
Copy link
Member

brmeyer commented May 19, 2014

Thanks @michael-schnell, seems reasonable. @sebersole, what do you think?

@michael-schnell
Copy link
Contributor Author

Any progress on merging the pull request?

@sebersole sebersole closed this Oct 29, 2014
@michael-schnell
Copy link
Contributor Author

No merge? What's wrong with the patch? Would at least be nice to add some kind of comment...

@sebersole
Copy link
Member

GitHub automatically closed Pull Requests targetting master when I was doing some moving around of branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet