CSHARP-3058: Obsolete ReIndex helper method.

Author: rstam

src/MongoDB.Driver.Core/Core/Operations/ReIndexOperation.cs

@@ -13,6 +13,7 @@
 * limitations under the License.
 */
 
+using System;
 using System.Threading;
 using System.Threading.Tasks;
 using MongoDB.Bson;
@@ -26,6 +27,7 @@ namespace MongoDB.Driver.Core.Operations
     /// <summary>
     /// Represents a reindex operation.
     /// </summary>
+    [Obsolete("This class will be removed in a later version of the driver.")]
     public class ReIndexOperation : IWriteOperation<BsonDocument>
     {
         // fields

src/MongoDB.Driver.Legacy/MongoCollection.cs

@@ -1833,14 +1833,17 @@ public ReadOnlyCollection<IEnumerator> ParallelScanAs(Type documentType, Paralle
         /// Runs the ReIndex command on this collection.
         /// </summary>
         /// <returns>A CommandResult.</returns>
+        [Obsolete("This method will be removed in a later version of the driver.")]
         public virtual CommandResult ReIndex()
         {
             return UsingImplicitSession(session => ReIndex(session));
         }
 
         private CommandResult ReIndex(IClientSessionHandle session)
         {
+#pragma warning disable 618
             var operation = new ReIndexOperation(_collectionNamespace, GetMessageEncoderSettings());
+#pragma warning restore 618
             var result = ExecuteWriteOperation(session, operation);
             return new CommandResult(result);
         }

tests/MongoDB.Driver.Core.Tests/Core/Operations/ReIndexOperationTests.cs

@@ -25,6 +25,7 @@
 
 namespace MongoDB.Driver.Core.Operations
 {
+#pragma warning disable 618
     public class ReIndexOperationTests : OperationTestBase
     {
         [Fact]
@@ -76,7 +77,7 @@ public void Execute_should_return_expected_result(
             [Values(false, true)]
             bool async)
         {
-            RequireServer.Check().ClusterTypes(ClusterType.Standalone, ClusterType.ReplicaSet);
+            RequireServer.Check().ClusterTypes(ClusterType.Standalone);
             EnsureCollectionExists();
             var subject = new ReIndexOperation(_collectionNamespace, _messageEncoderSettings);
 
@@ -111,17 +112,21 @@ public void Execute_should_throw_when_binding_is_null(
         public void Execute_should_send_session_id_when_supported(
             [Values(false, true)] bool async)
         {
-            RequireServer.Check().ClusterTypes(ClusterType.Standalone, ClusterType.ReplicaSet);
+            RequireServer.Check().ClusterTypes(ClusterType.Standalone);
             EnsureCollectionExists();
+#pragma warning disable 618
             var subject = new ReIndexOperation(_collectionNamespace, _messageEncoderSettings);
+#pragma warning restore 618
 
             VerifySessionIdWasSentWhenSupported(subject, "reIndex", async);
         }
 
         [Fact]
         public void CreateCommand_should_return_expected_result()
         {
+#pragma warning disable 618
             var subject = new ReIndexOperation(_collectionNamespace, _messageEncoderSettings);
+#pragma warning restore 618
 
             var result = subject.CreateCommand(null);
 
@@ -141,4 +146,5 @@ public void EnsureCollectionExists()
             ExecuteOperation(operation);
         }
     }
+#pragma warning restore 618
 }

tests/MongoDB.Driver.Legacy.Tests/MongoCollectionTests.cs

@@ -2989,7 +2989,7 @@ public void TestParallelScan()
         [Fact]
         public void TestReIndex()
         {
-            if (_primary.InstanceType != MongoServerInstanceType.ShardRouter)
+            if (_primary.InstanceType == MongoServerInstanceType.StandAlone)
             {
                 _collection.Drop();
                 _collection.Insert(new BsonDocument("x", 1));
@@ -2998,7 +2998,9 @@ public void TestReIndex()
                 // note: prior to 1.8.1 the reIndex command was returning duplicate ok elements
                 try
                 {
+#pragma warning disable 618
                     var result = _collection.ReIndex();
+#pragma warning restore 618
                     Assert.Equal(2, result.Response["nIndexes"].ToInt32());
                     Assert.Equal(2, result.Response["nIndexesWas"].ToInt32());
                 }