jOOQ / jOOQ/jOOQ

ParserCLI shouldn't be so newline sensitive for procedural logic

Open
#11,368 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

C: Functionality E: All Editions P: Medium T: Enhancement
Dominant language
Java
Stars
6.8k
Forks
1.2k
PR merge metrics
No merged PRs in 30d

Description

When translating procedural logic with ParserCLI, or statement batches containing ;, currently the ParserCLI behaviour is quite inconvenient as can be seen here:

image

Any ; character is interpreted as a delimiter, leading to parser errors. Solutions:

  • Implement DELIMITER and hint at how this works in the error message (e.g. DELIMITER // as in MySQL)
  • Improve the parser to recognise ; semantics

Perhaps this is just a simple bug. Let's investigate.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the procedural-logic or semicolon-containing statement batch through ParserCLI, using the example in the issue. Inspect how ParserCLI handles delimiters and parser errors, then determine whether delimiter support or improved semicolon recognition is appropriate. Done means procedural batches parse without errors and the error message explains any required delimiter syntax.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, sql
Domain
cli
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.