You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@grails.apache.org by GitBox <gi...@apache.org> on 2026/10/23 02:31:55 UTC
[GH] (grails-core/feat/sql-text-8.0.x): Workflow run "CI" failed!
The GitHub Actions job "CI" on grails-core.git/feat/sql-text-8.0.x has failed.
Run started by GitHub user borinquenkid (triggered by borinquenkid).
Head commit for run:
e79032ab7ab8310ae0ee067eb8ea6efa9cbb2594 / Walter Duque de Estrada <wb...@mac.com>
Hibernate 7: brackets group an expression unless they open an identifier, and native SQL is never read as HQL
From the review of #16735. The scanner read every [ in SQL as the start of a bracket identifier, so ARRAY['['], height was one column and the placeholders of ANY(ARRAY[?, ?]) were hidden, both of which worked before it. A [ now opens an identifier only where nothing to subscript precedes it: after an identifier, a closing parenthesis or a closing bracket it is structure, grouping like a parenthesis, so [box's width] stays one identifier while the string literals and placeholders inside an array literal stay visible. HQL groups brackets the same way and still has no bracket identifiers. HqlQueryContext.prepare read a native query as HQL, and HqlText lowercased with String.toLowerCase, which lengthens a dotted capital I and made the reader index the original past its end. Native SQL is no longer read at all, and the reader lowercases character by character so every index of the lowercase text is an index of the original. Covered through the criteria API (an array literal with a bracket
in a sqlProjection, placeholders inside an array in a sqlRestriction), through findAllWithSql with such a literal, and in the unit specs of SqlText, SqlProjection, HqlText and HqlQueryContext.
Report URL: https://github.com/apache/grails-core/actions/runs/39876372833
With regards,
GitHub Actions via GitBox