[SERVER-2579] change v8 wrapper to be lazy Created: 17/Feb/11 Updated: 12/Jul/16 Resolved: 02/May/11 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | JavaScript |
| Affects Version/s: | None |
| Fix Version/s: | 1.9.1 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Eliot Horowitz (Inactive) | Assignee: | Antoine Girbal |
| Resolution: | Done | Votes: | 2 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Participants: | |||||||||
| Description |
|
we need to make the BSON to v8 conversion lazy rather than aggressive, at least optionally. |
| Comments |
| Comment by free [ 31/Aug/11 ] |
|
Seems it won't depents on |
| Comment by auto [ 13/May/11 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: |
| Comment by auto [ 10/May/11 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: |
| Comment by auto [ 02/May/11 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: |
| Comment by auto [ 02/May/11 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: Also removed scope->setThis in favor of passing it in the invoke() command. |
| Comment by auto [ 25/Apr/11 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: |
| Comment by auto [ 25/Apr/11 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: |
| Comment by Antoine Girbal [ 19/Apr/11 ] |
|
Been doing some testing with wrapper and the performance is quite slower (2x) when using lazily populated arrays vs creating whole array ahead. |
| Comment by Antoine Girbal [ 06/Apr/11 ] |
|
so when I started v8 seemed to be quite slow, about 10x for a simple group.
This get the following results for mr1.js so far about 20-30% gain, potentially more as objects get much larger. |