linkedin / linkedin/dustjs

Performance issue with 2.7.4

Open
#751 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
2.9k
Forks
461
PR merge metrics
No merged PRs in 30d

Description

Hi,

I've found an performance issue in the current 2.7 branch and would fix that. I also found out that the code in master is already restructured and may not be affected anymore. Will the current master be merged in 2.7 or is it worth it to fix that 2.7 branch?

(v8 does not optimise that function)
https://github.com/linkedin/dustjs/blob/2.7/lib/parser.js#L268

Marc

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at lib/parser.js line 268 in the 2.7 branch and compare the corresponding code in master, since the issue says the parser was restructured there. Reproduce the reported V8 optimization problem and determine whether the 2.7 branch still needs a fix; done means the branch decision and performance impact are verified.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
web-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.