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

JPA Repository broken with SQLServer after Eclipselink update #1045

Closed
kaizimmerm opened this issue Nov 15, 2020 · 2 comments · Fixed by #1047
Closed

JPA Repository broken with SQLServer after Eclipselink update #1045

kaizimmerm opened this issue Nov 15, 2020 · 2 comments · Fixed by #1047
Labels

Comments

@kaizimmerm
Copy link
Contributor

kaizimmerm commented Nov 15, 2020

Hi,

the recent Eclipselink upgrade from #1038 unfortunately broke the SQLServer support.

It causes massive exceptions com.microsoft.sqlserver.jdbc.SQLServerException: ORDER BY items must appear in the select list if SELECT DISTINCT is specified.

The reason is the change introduced with eclipse-ee4j/eclipselink#300 which causes the error.

I am open for suggestions. We either drop SQLServer support or revert the Eclipselink upgrade.

I reported this in Eclipselink as well eclipse-ee4j/eclipselink#963

Kai

CC: @schabdo

@kaizimmerm kaizimmerm added the bug label Nov 15, 2020
@kaizimmerm kaizimmerm changed the title JPA Repository broken with SQLServer JPA Repository broken with SQLServer after Eclipselink update Nov 15, 2020
@schabdo
Copy link
Contributor

schabdo commented Nov 19, 2020

As far as I was able to track down the changes for SQL Server it seems that changes went straight into EclipseLink 2.7.4.

We either drop SQLServer support or revert the Eclipselink upgrade

Since we do not enforce Java 11 yet and there is still some way to go (e.g. official release, ...) I'd suggest to go for the latter option and downgrade EclipseLink for now. Furthermore there is already a fix (thanks a lot by the way!) handed-in to EclipseLink so it could be fixed in the meanwhile till we're good to go for Java 11

@kaizimmerm
Copy link
Contributor Author

Agree, did that already in #1047

This was referenced Nov 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants