This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 268745 - Fix performance in 'javascript2' code
Summary: Fix performance in 'javascript2' code
Status: NEW
Alias: None
Product: javascript
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: Dev
Hardware: PC Windows 7
: P2 normal (vote)
Assignee: Petr Pisl
URL:
Keywords:
Depends on: 268778 268980 268981
Blocks:
  Show dependency tree
 
Reported: 2016-10-30 13:42 UTC by NukemBy
Modified: 2016-11-14 22:10 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
javascript2-all-in-one-issues.zip (1.60 MB, application/x-zip-compressed)
2016-10-30 13:42 UTC, NukemBy
Details
js-patches.zip (40.65 KB, application/x-zip-compressed)
2016-10-31 14:37 UTC, NukemBy
Details

Note You need to log in before you can comment on or make changes to this bug.
Description NukemBy 2016-10-30 13:42:29 UTC
Created attachment 162689 [details]
javascript2-all-in-one-issues.zip

Hello, i'm using NetBeans for JS and Java devlopment. Thanks for new JS-related staff introduced in Release 8.2, but implementation has numerous performance issues - project is loaded for extremely long time and editing of files is laggy (while editing JS libraries of 10K lines).

I've self-profiled Netbeans and applied some hot fixies in my local dev-built. Most of the performance issues seem to be fixed - the rest does not look to be a show-stopper and do not look to be easily fixable by external person. Anyway - the build i have locally seems to be rather OK for me.

So ... 
- there are 20+ issues in attached zip each in separate file - the most severe typically go first. Each sub-folder includes screenshots from self profiler for the issue to be fixed. Also subfolder may or may not contain some description.

- zip also includes an all-one-patch - i was going to make individual patch per-issue, but this appears to be extremely time consuming for me

Note: it seems that enabled 'View -> Show Breadcrumbs' cause much more frequent reparsing of sources and self-profilier better shows hotspots in this case.


Patch relies on LruCache file, the new one i've added into 'openide.util\src\org\openide\util' - it is provided as separate file and already included in the patch referenced in another issue (https://netbeans.org/bugzilla/show_bug.cgi?id=268744 - i'm not sure if it will be aplied).

Attachment also includes test project with few 'real' files which provide enough load to easily reproduce issues.
Comment 1 NukemBy 2016-10-30 13:55:12 UTC
Note: Patch is made on-top of this revision http://hg.netbeans.org/main/rev/10cc566d91aa
Which i beleive is last commit for Release 8.2
Comment 2 Petr Pisl 2016-10-31 08:43:11 UTC
Thanks for this. I will go throuh one by one, evaluate and create separate issue for this.
Comment 3 NukemBy 2016-10-31 14:37:42 UTC
Created attachment 162702 [details]
js-patches.zip
Comment 4 NukemBy 2016-10-31 14:39:30 UTC
I'm using Git for local development - I've attached patches generated from Git for more granular changes (not sure if it is mercurial-compatible). This may be more helpful for "one by one" analysis.