-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Don't use SELECT DISTINCT when to_char() is used in a WHERE statement #22469
Don't use SELECT DISTINCT when to_char() is used in a WHERE statement #22469
Conversation
c489725
to
a8bd209
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure
So, my approach with group by didn't work:
So I tried casting there too:
So I tried to cast in the select and group by the alias:
One last try with:
If that also fails the only thing I can imagine is we just dont group on oracle |
Nice bug in Oracle Signed-off-by: Joas Schilling <[email protected]>
a8bd209
to
aa5e4aa
Compare
Worked, no more error in the log: |
/backport to stable19 |
/backport to stable18 |
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 32757: failuremariadb10.4-php7.3Show full log
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐘
Nice bug in Oracle
Found while debugging why Oracle tests failed installation in Notifications app:
nextcloud/notifications#700
Before