[SERVER-13812] Catch exception thrown by getShardsForQuery for geo query Created: 01/May/14 Updated: 11/Jul/16 Resolved: 02/May/14 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 2.6.2, 2.7.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Siyuan Zhou | Assignee: | Siyuan Zhou |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Operating System: | ALL | ||||
| Backport Completed: | |||||
| Participants: | |||||
| Linked BF Score: | 0 | ||||
| Description |
|
getShardsForQuery may throw exception with uassert, but write command should not throw. Add try catch around the call. |
| Comments |
| Comment by Daniel Pasette (Inactive) [ 16/May/14 ] |
|
I only backported the code change as the test code change depends on the test migration that happened in 2.7.0 and which will not be backported. |
| Comment by Githook User [ 16/May/14 ] |
|
Author: {u'username': u'visualzhou', u'name': u'Siyuan Zhou', u'email': u'siyuan.zhou@mongodb.com'}Message: |
| Comment by Githook User [ 06/May/14 ] |
|
Author: {u'username': u'visualzhou', u'name': u'Siyuan Zhou', u'email': u'siyuan.zhou@mongodb.com'}Message: |
| Comment by Matt Kangas [ 02/May/14 ] |
|
Note, there is a test improvement that we need to backport which makes this bug visible. Siyuan - please add info? |
| Comment by Githook User [ 02/May/14 ] |
|
Author: {u'username': u'visualzhou', u'name': u'Siyuan Zhou', u'email': u'siyuan.zhou@mongodb.com'}Message: |