[SERVER-1900] v8 Issue 708 Created: 06/Oct/10 Updated: 12/Jul/16 Resolved: 07/Oct/11 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | JavaScript |
| Affects Version/s: | None |
| Fix Version/s: | 2.1.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Aaron Staple | Assignee: | Antoine Girbal |
| Resolution: | Done | Votes: | 2 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Operating System: | ALL | ||||||||||||||||
| Participants: | |||||||||||||||||
| Description |
|
This jira is a placeholder for v8 Issue 708. Until Issue 708 is fixed, GetCurrentThreadId() will only return the correct id for a thread if no v8 thread has been started between the start of the current v8 thread and the call to GetCurrentThreadId() Our current implementation assumes Issue 708 has been fixed, a workaround for Issue 708 is possible but not ideal. |
| Comments |
| Comment by auto [ 15/Feb/12 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: |
| Comment by Antoine Girbal [ 07/Oct/11 ] |
|
Re-enabled tests, and tried with v8 r8669. |
| Comment by auto [ 06/Oct/11 ] |
|
Author: {u'login': u'agirbal', u'name': u'agirbal', u'email': u'antoine@10gen.com'}Message: - |
| Comment by Eliot Horowitz (Inactive) [ 24/Aug/11 ] |
|
@Ben - we're not looking at v8 again until after 2.0 ships. |
| Comment by Ben gamari [ 24/Aug/11 ] |
|
Ping. |
| Comment by Ben gamari [ 23/Jul/11 ] |
|
Judging by the fact that GetCurrentThreadId has been reworked around the Isolate class, there is a good chance this issue is fixed upstream. |
| Comment by Eliot Horowitz (Inactive) [ 14/Dec/10 ] |
|
submit patch to v8 |
| Comment by Aaron Staple [ 12/Oct/10 ] |
|
Issue 708 was filed in May 2010, and none of the v8 developers have commented on the issue since it was created. Last week I commented with a starting point for a potential fix, and there have been no responses. If we are interested in resolving 1) Submit a proper v8 patch and hope it is accepted As a reminder, |
| Comment by Eliot Horowitz (Inactive) [ 07/Oct/10 ] |
|
put here since we're blocking on v8 change |
| Comment by auto [ 06/Oct/10 ] |
|
Author: {'login': 'astaple', 'name': 'Aaron', 'email': 'aaron@10gen.com'}Message: |
| Comment by auto [ 06/Oct/10 ] |
|
Author: {'login': 'astaple', 'name': 'Aaron', 'email': 'aaron@10gen.com'}Message: |