Jira (PDB-5260) Fix PQL parse performance problems

10 views
Skip to first unread message

Rob Browning (Jira)

unread,
Sep 8, 2021, 12:35:03 PM9/8/21
to puppe...@googlegroups.com
Rob Browning created an issue
 
PuppetDB / Improvement PDB-5260
Fix PQL parse performance problems
Issue Type: Improvement Improvement
Assignee: Unassigned
Created: 2021/09/08 9:34 AM
Priority: Normal Normal
Reporter: Rob Browning

We've discovered that PQL parsing (via instaparse) can be extremely slow (say 20s for a pql query with ~1000 filter clauses). A brief investigation suggests that this may be a known issue with instaparse.

It's not yet clear whether we might be able to address this by rearranging our grammar somehow, or we might want to rewrite the parser (either via manual recursive descent, or via something like antlr, flex/bison, etc.

Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.13.2#813002-sha1:c495a97)
Atlassian logo

Rob Browning (Jira)

unread,
Sep 8, 2021, 12:38:03 PM9/8/21
to puppe...@googlegroups.com
Rob Browning updated an issue
Change By: Rob Browning
Component/s: PuppetDB

Austin Blatt (Jira)

unread,
Sep 8, 2021, 12:40:03 PM9/8/21
to puppe...@googlegroups.com
Austin Blatt updated an issue
Change By: Austin Blatt
Component/s: PuppetDB

Rob Browning (Jira)

unread,
Sep 8, 2021, 12:46:02 PM9/8/21
to puppe...@googlegroups.com
Rob Browning updated an issue
Change By: Rob Browning
Sprint: HAHA/Grooming

David McTavish (Jira)

unread,
Jan 12, 2022, 3:53:02 PM1/12/22
to puppe...@googlegroups.com
David McTavish updated an issue
Change By: David McTavish
Epic Link: PE-31891
This message was sent by Atlassian Jira (v8.20.2#820002-sha1:829506d)
Atlassian logo

David McTavish (Jira)

unread,
Feb 9, 2022, 2:53:02 PM2/9/22
to puppe...@googlegroups.com
David McTavish updated an issue
Change By: David McTavish
Sprint: HAHA/Grooming

David McTavish (Jira)

unread,
Feb 10, 2022, 10:09:02 AM2/10/22
to puppe...@googlegroups.com
David McTavish updated an issue
Change By: David McTavish
Priority: Normal Low
Reply all
Reply to author
Forward
0 new messages