Skip to content

Commit 743ad48

Browse files
committed
Small comments
1 parent 3e59f5a commit 743ad48

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/MongoDB.Driver/CreateCollectionOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ public bool? NoPadding
127127
set { _noPadding = value; }
128128
}
129129

130-
//DOMAIN-API We need to remove this, and have only the SerializationDomain property.
130+
//DOMAIN-API We need to remove this, and have only the SerializationDomain property. When we have builder, we will add Obsolete
131131
//We should also decide if we even need any of those two properties.
132132
/// <summary>
133133
/// Gets or sets the serializer registry.

src/MongoDB.Driver/MongoDatabase.cs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -668,8 +668,11 @@ private IWriteOperation<BsonDocument> CreateCreateCollectionOperation<TDocument>
668668
new(documentSerializer, serializerRegistry, translationOptions: translationOptions) :
669669
new(documentSerializer, options.SerializationDomain, translationOptions: translationOptions);
670670

671+
//var serializationDomain = Settings.SerializationDomain.WithSerializerRegistry(serializerRegistry); //FP This could be the way to go
672+
671673
//DOMAIN-API This will need to go away
672-
var clusteredIndex = options.SerializationDomain is null? options.ClusteredIndex?.Render(documentSerializer, serializerRegistry, translationOptions):
674+
var clusteredIndex = options.SerializationDomain is null?
675+
options.ClusteredIndex?.Render(documentSerializer, serializerRegistry, translationOptions):
673676
options.ClusteredIndex?.Render(documentSerializer, options.SerializationDomain, translationOptions);
674677

675678
var validator = options.Validator?.Render(renderArgs);

0 commit comments

Comments
 (0)