diff options
author | Samantha Ritter <samantha.ritter@10gen.com> | 2012-08-08 10:21:53 -0400 |
---|---|---|
committer | Samantha Ritter <samantha.ritter@10gen.com> | 2012-08-08 10:21:53 -0400 |
commit | 12f558d6ae8196fae38d60bc22f333f02e298641 (patch) | |
tree | f2779a0f2d0bf876b8cf458f3b1653c0fe7c776a | |
parent | bc1bd6b98c18cc38222364fc90123ea14189d170 (diff) | |
download | mongo-12f558d6ae8196fae38d60bc22f333f02e298641.tar.gz |
SERVER--6708 sh._adminCommand is silent, and sh helpers return result object
-rw-r--r-- | src/mongo/shell/utils_sh.js | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/src/mongo/shell/utils_sh.js b/src/mongo/shell/utils_sh.js index 1ec5da09df2..3aa2102b2f4 100644 --- a/src/mongo/shell/utils_sh.js +++ b/src/mongo/shell/utils_sh.js @@ -13,13 +13,7 @@ sh._checkFullName = function( fullName ) { sh._adminCommand = function( cmd , skipCheck ) { if ( ! skipCheck ) sh._checkMongos(); - var res = db.getSisterDB( "admin" ).runCommand( cmd ); - - if ( res == null || ! res.ok ) { - print( "command failed: " + tojson( res ) ) - } - - return res; + return db.getSisterDB( "admin" ).runCommand( cmd ); } sh._dataFormat = function( bytes ){ @@ -62,12 +56,12 @@ sh.status = function( verbose , configDB ) { } sh.addShard = function( url ){ - sh._adminCommand( { addShard : url } , true ) + return sh._adminCommand( { addShard : url } , true ); } sh.enableSharding = function( dbname ) { assert( dbname , "need a valid dbname" ) - sh._adminCommand( { enableSharding : dbname } ) + return sh._adminCommand( { enableSharding : dbname } ); } sh.shardCollection = function( fullName , key , unique ) { @@ -79,17 +73,17 @@ sh.shardCollection = function( fullName , key , unique ) { if ( unique ) cmd.unique = true; - sh._adminCommand( cmd ) + return sh._adminCommand( cmd ); } sh.splitFind = function( fullName , find ) { sh._checkFullName( fullName ) - sh._adminCommand( { split : fullName , find : find } ) + return sh._adminCommand( { split : fullName , find : find } ); } sh.splitAt = function( fullName , middle ) { sh._checkFullName( fullName ) - sh._adminCommand( { split : fullName , middle : middle } ) + return sh._adminCommand( { split : fullName , middle : middle } ); } sh.moveChunk = function( fullName , find , to ) { |