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.

Sign up for GitHub

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

Jump to bottom

Parsing error of Execution Time and JIT time because of double quptes #573

Closed
Krysztophe opened this issue Mar 2, 2023 · 0 comments · Fixed by #656
Closed

Parsing error of Execution Time and JIT time because of double quptes #573

Krysztophe opened this issue Mar 2, 2023 · 0 comments · Fixed by #656

Comments

@Krysztophe
Copy link

pev2 correctly formats plans with leading and trailing double quotes,
but there is a problem in some cases, where the Execution time and the JIT time are not computed.

Example: https://explain.dalibo.com/plan/2e9h69cgh629f36e

This works when removing all " : https://explain.dalibo.com/plan/b9b6eg9f61e9eh4f

(The plan was massively edited for confidentiality but that does not change the problem; real plans in private if needed).
pgiraud added a commit that referenced this issue Aug 21, 2024
@pgiraud
Fix parsing issue for last line if contains quote
09bee7a 
For example, the "Execution time" line was not parsed correctly when
surrounded by quotes and on the very last line (no line break after)

Fixes #573
This was referenced Aug 21, 2024
Merged
Closed
pgiraud added a commit that referenced this issue Aug 21, 2024
@pgiraud
Fix parsing issue for last line if contains quote
1264b73 
For example, the "Execution time" line was not parsed correctly when
surrounded by quotes and on the very last line (no line break after)

Fixes #573
Fixes #601
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix parsing issue for last line if contains quote dalibo/pev2
1 participant
@Krysztophe