Skip to content

Commit e049234

Browse files
committed
Make a few minor changes
JAVA-5527
1 parent cf46b46 commit e049234

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

driver-core/src/main/com/mongodb/client/result/bulk/ClientBulkWriteResult.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,6 @@ public interface ClientBulkWriteResult {
8686
*/
8787
long getModifiedCount() throws UnsupportedOperationException;
8888

89-
// BULK-TODO Is ReplaceOne reported as 1 modified (I expect this behavior), or 1 deleted and 1 inserted?
90-
9189
/**
9290
* The number of documents that were deleted across all delete operations.
9391
*

driver-core/src/main/com/mongodb/internal/client/result/bulk/AcknowledgedVerboseClientBulkWriteResult.java

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,10 @@
1919
import com.mongodb.client.result.bulk.ClientDeleteResult;
2020
import com.mongodb.client.result.bulk.ClientInsertOneResult;
2121
import com.mongodb.client.result.bulk.ClientUpdateResult;
22-
import com.mongodb.lang.Nullable;
2322

2423
import java.util.Map;
2524
import java.util.Objects;
2625

27-
import static java.util.Collections.emptyMap;
2826
import static java.util.Collections.unmodifiableMap;
2927

3028
/**
@@ -38,13 +36,13 @@ public final class AcknowledgedVerboseClientBulkWriteResult implements ClientBul
3836

3937
public AcknowledgedVerboseClientBulkWriteResult(
4038
final AcknowledgedSummaryClientBulkWriteResult summaryResults,
41-
@Nullable final Map<Long, ClientInsertOneResult> insertResults,
42-
@Nullable final Map<Long, ClientUpdateResult> updateResults,
43-
@Nullable final Map<Long, ClientDeleteResult> deleteResults) {
39+
final Map<Long, ClientInsertOneResult> insertResults,
40+
final Map<Long, ClientUpdateResult> updateResults,
41+
final Map<Long, ClientDeleteResult> deleteResults) {
4442
this.summaryResults = summaryResults;
45-
this.insertResults = insertResults == null ? emptyMap() : unmodifiableMap(insertResults);
46-
this.updateResults = updateResults == null ? emptyMap() : unmodifiableMap(updateResults);
47-
this.deleteResults = deleteResults == null ? emptyMap() : unmodifiableMap(deleteResults);
43+
this.insertResults = unmodifiableMap(insertResults);
44+
this.updateResults = unmodifiableMap(updateResults);
45+
this.deleteResults = unmodifiableMap(deleteResults);
4846
}
4947

5048
@Override

0 commit comments

Comments
 (0)