Skip to content

jsqlparser 5.4.162, guard reads blank lines as whitespace - #66

Open
stbischof wants to merge 1 commit into
mainfrom
update/jsqlparser-5.4.162
Open

stbischof wants to merge 1 commit into
mainfrom
update/jsqlparser-5.4.162

Conversation

@stbischof

Copy link
Copy Markdown
Contributor
  • jsqlparser 5.4.162 (manticore build): parse() rejects text after the first statement (#2681, fixed upstream in fix: require end of input when parsing a single statement JSQLParser/JSqlParser#2733). jsqltranspiler stays at 1.13, the latest release.
  • guard: runs of blank lines outside literals and comments are collapsed before parsing. JSqlParser ends a statement at two blank lines, so select *\n\n\nfrom foo was refused as two statements.
  • DeparserColumResolver.getResolvedStatementText(String) parses a script and refuses more than one statement instead of dropping the rest.

@stbischof

Copy link
Copy Markdown
Contributor Author

Waiting for a jsqltranspiler release. Versions on Maven Central (2026-10-16):

artifact latest built against
ai.starlake.jsqltranspiler:jsqltranspiler 1.13 (2026-09-18) com.manticore-projects.jsqlformatter:jsqlparser 5.4.15
com.manticore-projects.jsqlformatter:jsqlparser 5.4.162 (2026-10-14)

5.4.162 no longer has AllColumns.getExceptColumns() returning ExpressionList, so jsqltranspiler 1.13 fails with NoSuchMethodError at JSQLExpressionColumnResolver.java:354 (Test guard/jsqltranspiler).
The guard fix alone passes on 5.4.15 (303 tests, local). Keep open until jsqltranspiler is released against a jsqlparser that has #2733.

@stbischof

Copy link
Copy Markdown
Contributor Author

Release asked for in starlake-ai/jsqltranspiler#154. The guard fix moved to #68, so once that is merged this PR keeps only the version update.

parse() now rejects text after the first statement (JSqlParser #2681).

Signed-off-by: Stefan Bischof <stbischof@bipolis.org>
@stbischof
stbischof force-pushed the update/jsqlparser-5.4.162 branch from 104c5a6 to 29dfba4 Compare October 16, 2026 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant