Skip to content

Commit 71107b6

Browse files
committed
CSHARP-1146: Add IsAcknowledged to WriteConcern and deprecate Enabled.
1 parent 49aa14c commit 71107b6

19 files changed

+58
-49
lines changed

MongoDB.Driver/MongoConnectionStringBuilder.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -446,7 +446,7 @@ public SafeMode SafeMode
446446
var writeConcern = value.WriteConcern;
447447
FSync = writeConcern.FSync;
448448
Journal = writeConcern.Journal;
449-
W = writeConcern.W ?? (writeConcern.Enabled ? 1 : 0);
449+
W = writeConcern.W ?? (writeConcern.IsAcknowledged ? 1 : 0);
450450
WTimeout = writeConcern.WTimeout;
451451
}
452452
}

MongoDB.Driver/MongoUrlBuilder.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -402,7 +402,7 @@ public SafeMode SafeMode
402402
var writeConcern = value.WriteConcern;
403403
FSync = writeConcern.FSync;
404404
Journal = writeConcern.Journal;
405-
W = writeConcern.W ?? (writeConcern.Enabled ? 1 : 0);
405+
W = writeConcern.W ?? (writeConcern.IsAcknowledged ? 1 : 0);
406406
WTimeout = writeConcern.WTimeout;
407407
}
408408
}

MongoDB.Driver/Operations/BulkMixedWriteOperation.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ public BulkWriteResult Execute(MongoConnection connection)
101101
throw new InvalidOperationException("Bulk write operation is empty.");
102102
}
103103

104-
var combiner = new BulkWriteBatchResultCombiner(batchResults, _writeConcern.Enabled);
104+
var combiner = new BulkWriteBatchResultCombiner(batchResults, _writeConcern.IsAcknowledged);
105105
return combiner.CreateResultOrThrowIfHasErrors(remainingRequests);
106106
}
107107

MongoDB.Driver/Operations/BulkUnmixedWriteOperationBase.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ public virtual BulkWriteResult Execute(MongoConnection connection)
7171
}
7272
}
7373

74-
var combiner = new BulkWriteBatchResultCombiner(batchResults, _args.WriteConcern.Enabled);
74+
var combiner = new BulkWriteBatchResultCombiner(batchResults, _args.WriteConcern.IsAcknowledged);
7575
return combiner.CreateResultOrThrowIfHasErrors(remainingRequests);
7676
}
7777

MongoDB.Driver/Operations/BulkUnmixedWriteOperationEmulatorBase.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public BulkWriteResult Execute(MongoConnection connection)
5959
originalIndex++;
6060
}
6161

62-
var combiner = new BulkWriteBatchResultCombiner(batchResults, _args.WriteConcern.Enabled);
62+
var combiner = new BulkWriteBatchResultCombiner(batchResults, _args.WriteConcern.IsAcknowledged);
6363
return combiner.CreateResultOrThrowIfHasErrors(remainingRequests);
6464
}
6565

MongoDB.Driver/Operations/DeleteOpcodeOperation.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public DeleteOpcodeOperation(BulkDeleteOperationArgs args)
3636
public WriteConcernResult Execute(MongoConnection connection)
3737
{
3838
var serverInstance = connection.ServerInstance;
39-
if (serverInstance.Supports(FeatureId.WriteCommands) && _args.WriteConcern.Enabled)
39+
if (serverInstance.Supports(FeatureId.WriteCommands) && _args.WriteConcern.IsAcknowledged)
4040
{
4141
var emulator = new DeleteOpcodeOperationEmulator(_args);
4242
return emulator.Execute(connection);
@@ -68,7 +68,7 @@ public WriteConcernResult Execute(MongoConnection connection)
6868
sendMessageResult = SendMessageWithWriteConcern(connection, buffer, message.RequestId, ReaderSettings, WriterSettings, WriteConcern);
6969
}
7070

71-
return WriteConcern.Enabled ? ReadWriteConcernResult(connection, sendMessageResult) : null;
71+
return WriteConcern.IsAcknowledged ? ReadWriteConcernResult(connection, sendMessageResult) : null;
7272
}
7373
}
7474
}

MongoDB.Driver/Operations/DeleteOpcodeOperationEmulator.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ public WriteConcernResult Execute(MongoConnection connection)
6262
}
6363
else
6464
{
65-
if (_args.WriteConcern.Enabled)
65+
if (_args.WriteConcern.IsAcknowledged)
6666
{
6767
return converter.ToWriteConcernResult(bulkWriteResult);
6868
}

MongoDB.Driver/Operations/InsertOpcodeOperation.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,13 +40,13 @@ public InsertOpcodeOperation(BulkInsertOperationArgs args)
4040
public IEnumerable<WriteConcernResult> Execute(MongoConnection connection)
4141
{
4242
var serverInstance = connection.ServerInstance;
43-
if (serverInstance.Supports(FeatureId.WriteCommands) && _args.WriteConcern.Enabled)
43+
if (serverInstance.Supports(FeatureId.WriteCommands) && _args.WriteConcern.IsAcknowledged)
4444
{
4545
var emulator = new InsertOpcodeOperationEmulator(_args);
4646
return emulator.Execute(connection);
4747
}
4848

49-
var results = WriteConcern.Enabled ? new List<WriteConcernResult>() : null;
49+
var results = WriteConcern.IsAcknowledged ? new List<WriteConcernResult>() : null;
5050
var finalException = (Exception)null;
5151

5252
var requests = _args.Requests.Cast<InsertRequest>();
@@ -100,7 +100,7 @@ public IEnumerable<WriteConcernResult> Execute(MongoConnection connection)
100100
{
101101
finalException = ex;
102102
}
103-
else if (WriteConcern.Enabled)
103+
else if (WriteConcern.IsAcknowledged)
104104
{
105105
results.Add(writeConcernResult);
106106
ex.Data["results"] = results;
@@ -122,7 +122,7 @@ public IEnumerable<WriteConcernResult> Execute(MongoConnection connection)
122122
}
123123
}
124124

125-
if (WriteConcern.Enabled && finalException != null)
125+
if (WriteConcern.IsAcknowledged && finalException != null)
126126
{
127127
finalException.Data["results"] = results;
128128
throw finalException;
@@ -135,7 +135,7 @@ public IEnumerable<WriteConcernResult> Execute(MongoConnection connection)
135135
private SendMessageWithWriteConcernResult SendBatch(MongoConnection connection, BsonBuffer buffer, int requestId, bool isLast)
136136
{
137137
var writeConcern = WriteConcern;
138-
if (!writeConcern.Enabled && !_continueOnError && !isLast)
138+
if (!writeConcern.IsAcknowledged && !_continueOnError && !isLast)
139139
{
140140
writeConcern = WriteConcern.Acknowledged;
141141
}

MongoDB.Driver/Operations/InsertOpcodeOperationEmulator.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ public IEnumerable<WriteConcernResult> Execute(MongoConnection connection)
6565
}
6666
else
6767
{
68-
if (_args.WriteConcern.Enabled)
68+
if (_args.WriteConcern.IsAcknowledged)
6969
{
7070
return new[] { converter.ToWriteConcernResult(bulkWriteResult) };
7171
}

MongoDB.Driver/Operations/UpdateOpcodeOperation.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public UpdateOpcodeOperation(BulkUpdateOperationArgs args)
3636
public WriteConcernResult Execute(MongoConnection connection)
3737
{
3838
var serverInstance = connection.ServerInstance;
39-
if (serverInstance.Supports(FeatureId.WriteCommands) && _args.WriteConcern.Enabled)
39+
if (serverInstance.Supports(FeatureId.WriteCommands) && _args.WriteConcern.IsAcknowledged)
4040
{
4141
var emulator = new UpdateOpcodeOperationEmulator(_args);
4242
return emulator.Execute(connection);
@@ -65,7 +65,7 @@ public WriteConcernResult Execute(MongoConnection connection)
6565
sendMessageResult = SendMessageWithWriteConcern(connection, buffer, message.RequestId, ReaderSettings, WriterSettings, WriteConcern);
6666
}
6767

68-
return WriteConcern.Enabled ? ReadWriteConcernResult(connection, sendMessageResult) : null;
68+
return WriteConcern.IsAcknowledged ? ReadWriteConcernResult(connection, sendMessageResult) : null;
6969
}
7070
}
7171
}

0 commit comments

Comments
 (0)