[SERVER-7017] Rename of DB can cause overflow of namespace length Created: 12/Sep/12 Updated: 11/Jul/16 Resolved: 16/Oct/12 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Index Maintenance |
| Affects Version/s: | 2.0.7, 2.2.0 |
| Fix Version/s: | 2.3.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | David Hows | Assignee: | David Hows |
| Resolution: | Done | Votes: | 8 |
| Labels: | buildbot, neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Operating System: | ALL | ||||||||
| Participants: | |||||||||
| Description |
|
If you rename a collection such that the length of the full collection namespace + the name of the index is > 128 characters you will get a namespace error. The move of the collection will succeed but you will be left with an index floating that cannot be removed. Have attached a set of commands which will create a stuck index for a collection which no longer exists. |
| Comments |
| Comment by auto [ 16/Oct/12 ] |
|
Author: {u'date': u'2012-10-15T15:00:10-07:00', u'email': u'tad@10gen.com', u'name': u'Tad Marshall'}Message: Revert jstests/rename2.js to the way it was before, add rename6.js |
| Comment by Tad Marshall [ 15/Oct/12 ] |
|
I talked to Greg about this. Since this test is exercising functionality that was not present in older versions, it needs to be added to one of the lists in multi_version_sharding_passthrough.js . Probably the first list in the file, "testsToIgnore", is the right one for it. Probably a better option is to split the rename2.js file into two files; one would be the old version of this file (that will pass the multi-version test) and one will be the new test that will not pass in version 2.0. This way we don't lose the old test for multi-version testing. |
| Comment by Eric Milkie [ 15/Oct/12 ] |
|
The code submission broke the multiversion unit tests as part of multi_version_sharing_passthrough.js: |
| Comment by auto [ 15/Oct/12 ] |
|
Author: {u'date': u'2012-10-15T07:14:51-07:00', u'name': u'Tad Marshall', u'email': u'tad@10gen.com'}Message: |
| Comment by auto [ 15/Oct/12 ] |
|
Author: {u'date': u'2012-10-15T06:40:49-07:00', u'name': u'Ian Whalen', u'email': u'ian.whalen@gmail.com'}Message: Merge pull request #315 from daveh86/ Fix for |
| Comment by auto [ 15/Oct/12 ] |
|
Author: {u'date': u'2012-10-15T06:40:49-07:00', u'name': u'Ian Whalen', u'email': u'ian.whalen@gmail.com'}Message: Merge pull request #315 from daveh86/ Fix for |
| Comment by auto [ 15/Oct/12 ] |
|
Author: {u'date': u'2012-10-04T20:39:23-07:00', u'name': u'daveh86', u'email': u'howsdav@gmail.com'}Message: Fix for |
| Comment by Eliot Horowitz (Inactive) [ 21/Sep/12 ] |
|
We cannot truncate index names, so should just fail fast. |