[JAVA-1165] Some DBCursor array methods may not close underlying cursor Created: 10/Apr/14 Updated: 01/Aug/14 Resolved: 16/Apr/14 |
|
| Status: | Closed |
| Project: | Java Driver |
| Component/s: | Query Operations |
| Affects Version/s: | 2.12.0 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | John Morales | Assignee: | John Morales |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
It appears that 4 methods exposed on DBCursor have the potential to not have the underlying cursor closed in the event of an IO error.
Note that exhausting a cursor causes the server-side cursor to be closed. Therefore on the one hand, 3 out of 4 methods above are fine on the happy path, and are only exposed in the case of an IO error. However toArray(int) is more risky, since in this case the cursor may not be exhausted, either, depending on the int argument value and length of the result set. |
| Comments |
| Comment by Githook User [ 01/Aug/14 ] |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: |
| Comment by Githook User [ 01/Aug/14 ] |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: |
| Comment by Githook User [ 01/Aug/14 ] |
|
Author: {u'username': u'john-morales', u'name': u'John Morales', u'email': u'john.morales@10gen.com'}Message: |
| Comment by Githook User [ 25/Apr/14 ] |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: |
| Comment by Githook User [ 16/Apr/14 ] |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: |
| Comment by Jeffrey Yemin [ 16/Apr/14 ] |
|
Backed out the change, after discussing and determining it's too risky. |
| Comment by Jeffrey Yemin [ 16/Apr/14 ] |
|
Need to merge this in to 3.0.x branch |
| Comment by Githook User [ 16/Apr/14 ] |
|
Author: {u'username': u'john-morales', u'name': u'John Morales', u'email': u'john.morales@10gen.com'}Message: |