Skip to content

HHH 11544 - Test and fix #1844

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

Closed
wants to merge 3 commits into from
Closed

HHH 11544 - Test and fix #1844

wants to merge 3 commits into from

Conversation

beikov
Copy link
Member

@beikov beikov commented Mar 11, 2017

@dreab8
Copy link
Member

dreab8 commented Mar 13, 2017

org.hibernate.test.inheritance.discriminator.MultiSingleTableLoadTest is failing, a wrong SQL query is generated.

@beikov
Copy link
Member Author

beikov commented Sep 1, 2017

This should do it, although I haven't run the full testsuite with it.

@gbadner
Copy link
Contributor

gbadner commented Sep 12, 2017

Before rebasing master these test fails:
org.hibernate.test.inheritance.discriminator.MultiInheritanceImplicitDowncastTest#testQueryConstruction
org.hibernate.test.annotations.genericsinheritance.GenericsInheritanceTest#classMethod

After rebasing master, the first test passes, but the second test still fails.

@beikov
Copy link
Member Author

beikov commented Sep 13, 2017

I'd give it a look on the weekend. If that is too late for the release, just move it to the version please.

@Naros
Copy link
Member

Naros commented Sep 13, 2017

No problem @beikov, I've gone ahead and pushed it out to the next release cycle.

@beikov
Copy link
Member Author

beikov commented Sep 15, 2017

Huh, I'm not seeing any of these failures locally after rebasing. How did you invoke the build @gbadner ?

@gbadner
Copy link
Contributor

gbadner commented Oct 19, 2017

I'm not seeing any failures now.

@vladmihalcea
Copy link
Contributor

Applied upstream, thanks

@beikov beikov deleted the HHH-11544-2 branch May 22, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants