summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAdam Midvidy <amidvidy@gmail.com>2015-05-11 09:00:31 -0400
committerAdam Midvidy <amidvidy@gmail.com>2015-05-11 09:00:31 -0400
commitb73ab5765fb75ae87ee5ad0f4afbc4fdfc2bc151 (patch)
tree0712d33d484bfeafeb779de7eaa7b2f8b22aaaea
parente45ec68181c5d27074c75be6627f9660f75cf484 (diff)
downloadmongo-b73ab5765fb75ae87ee5ad0f4afbc4fdfc2bc151.tar.gz
Revert "SERVER-16741 Add escape sequence to directoryperdb paths in KV Store"
This reverts commit e45ec68181c5d27074c75be6627f9660f75cf484.
-rw-r--r--src/mongo/db/namespace_string.cpp44
-rw-r--r--src/mongo/db/namespace_string.h5
-rw-r--r--src/mongo/db/storage/kv/kv_catalog.cpp2
3 files changed, 1 insertions, 50 deletions
diff --git a/src/mongo/db/namespace_string.cpp b/src/mongo/db/namespace_string.cpp
index fe50e2039be..d4637695ee7 100644
--- a/src/mongo/db/namespace_string.cpp
+++ b/src/mongo/db/namespace_string.cpp
@@ -34,38 +34,6 @@ namespace mongo {
using std::string;
- /* A map of characters to escape. Instead of printing certain characters we output
- * based on the following table.
- */
- static const std::string escapeTable[256] = {
- ".00", ".01", ".02", ".03", ".04", ".05", ".06", ".07", ".08", ".09",
- ".10", ".11", ".12", ".13", ".14", ".15", ".16", ".17", ".18", ".19",
- ".20", ".21", ".22", ".23", ".24", ".25", ".26", ".27", ".28", ".29",
- ".30", ".31", ".32", ".33", ".34", ".35", ".36", ".37", ".38", ".39",
- ".40", ".41", ".42", ".43", ".44", ".45", ".", ".47", "0", "1",
- "2", "3", "4", "5", "6", "7", "8", "9", ".58", ".59",
- ".60", ".61", ".62", ".63", ".64", "A", "B", "C", "D", "E",
- "F", "G", "H", "I", "J", "K", "L", "M", "N", "O",
- "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Y",
- "Z", ".91", ".92", ".93", ".94", "_", ".96", "a", "b", "c",
- "d", "e", "f", "g", "h", "i", "j", "k", "l", "m",
- "n", "o", "p", "q", "r", "s", "t", "u", "v", "w",
- "x", "y", "z", ".123", ".124", ".125", ".126", ".127", ".128", ".129",
- ".130", ".131", ".132", ".133", ".134", ".135", ".136", ".137", ".138", ".139",
- ".140", ".141", ".142", ".143", ".144", ".145", ".146", ".147", ".148", ".149",
- ".150", ".151", ".152", ".153", ".154", ".155", ".156", ".157", ".158", ".159",
- ".160", ".161", ".162", ".163", ".164", ".165", ".166", ".167", ".168", ".169",
- ".170", ".171", ".172", ".173", ".174", ".175", ".176", ".177", ".178", ".179",
- ".180", ".181", ".182", ".183", ".184", ".185", ".186", ".187", ".188", ".189",
- ".190", ".191", ".192", ".193", ".194", ".195", ".196", ".197", ".198", ".199",
- ".200", ".201", ".202", ".203", ".204", ".205", ".206", ".207", ".208", ".209",
- ".210", ".211", ".212", ".213", ".214", ".215", ".216", ".217", ".218", ".219",
- ".220", ".221", ".222", ".223", ".224", ".225", ".226", ".227", ".228", ".229",
- ".230", ".231", ".232", ".233", ".234", ".235", ".236", ".237", ".238", ".239",
- ".240", ".241", ".242", ".243", ".244", ".245", ".246", ".247", ".248", ".249",
- ".250", ".251", ".252", ".253", ".254", ".255"
- };
-
bool legalClientSystemNS( StringData ns , bool write ) {
if( ns == "local.system.replset" ) return true;
@@ -100,16 +68,4 @@ namespace mongo {
return NamespaceString(db(), coll().substr(listIndexesGetMoreNSPrefix.size()));
}
- string NamespaceString::escapeDbName( const StringData dbname ) {
- std::string escapedDbName;
-
- // pre-alloc the return string as it will always be the same as dbname at a minimum.
- escapedDbName.reserve(dbname.size());
-
- for (unsigned char c : dbname) {
- escapedDbName += escapeTable[c];
- }
- return escapedDbName;
- }
-
}
diff --git a/src/mongo/db/namespace_string.h b/src/mongo/db/namespace_string.h
index 655dbf68a73..72f357f13c3 100644
--- a/src/mongo/db/namespace_string.h
+++ b/src/mongo/db/namespace_string.h
@@ -145,11 +145,6 @@ namespace mongo {
std::string getCommandNS() const;
/**
- * Function to escape most non-alpha characters from file names
- */
- static std::string escapeDbName( const StringData dbname );
-
- /**
* @return true if ns is 'normal'. A "$" is used for namespaces holding index data,
* which do not contain BSON objects in their records. ("oplog.$main" is the exception)
*/
diff --git a/src/mongo/db/storage/kv/kv_catalog.cpp b/src/mongo/db/storage/kv/kv_catalog.cpp
index b394e942fd2..764e4b4c93e 100644
--- a/src/mongo/db/storage/kv/kv_catalog.cpp
+++ b/src/mongo/db/storage/kv/kv_catalog.cpp
@@ -122,7 +122,7 @@ namespace {
// If this changes to not put _rand at the end, _hasEntryCollidingWithRand will need fixing.
StringBuilder buf;
if ( _directoryPerDb ) {
- buf << NamespaceString::escapeDbName( nsToDatabaseSubstring( ns ) ) << '/';
+ buf << nsToDatabaseSubstring( ns ) << '/';
}
buf << kind;
buf << ( _directoryForIndexes ? '/' : '-' );