[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: File RecreateIssue    
Issue Links:
Related
related to SERVER-7282 Logic for checking namespace name len... Closed
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: SERVER-7017 split rename2.js into two tests for multiVersion

Revert jstests/rename2.js to the way it was before, add rename6.js
for the new tests, make multi_version_sharding_passthrough.js skip
rename6.js since it is new functionality.
Branch: master
https://github.com/mongodb/mongo/commit/309cf5f83fa75357a89844e42923a0c907f52edc

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:
http://buildbot.mongodb.org/builders/Linux%2064-bit%20multiVersion/builds/402

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: SERVER-7017 fix duplicate ID
Branch: master
https://github.com/mongodb/mongo/commit/b525bc684b8b1277228249c30a7db9fad5dece76

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/SERVER-7017

Fix for SERVER-7017
Branch: master
https://github.com/mongodb/mongo/commit/ce7dcaea452ad5c17d8901154f7d23b78f10430c

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/SERVER-7017

Fix for SERVER-7017
Branch: master
https://github.com/mongodb/mongo/commit/ce7dcaea452ad5c17d8901154f7d23b78f10430c

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 SERVER-7017
Branch: master
https://github.com/mongodb/mongo/commit/5c18832af5f97b52826098d6f71f29531a50f614

Comment by Eliot Horowitz (Inactive) [ 21/Sep/12 ]

We cannot truncate index names, so should just fail fast.

Generated at Thu Feb 08 03:13:24 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.