Skip to content

Commit 7634e48

Browse files
committed
Small comments
1 parent 17031c1 commit 7634e48

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
@@ -128,7 +128,7 @@ public bool? NoPadding
128128
set { _noPadding = value; }
129129
}
130130

131-
//DOMAIN-API We need to remove this, and have only the SerializationDomain property.
131+
//DOMAIN-API We need to remove this, and have only the SerializationDomain property. When we have builder, we will add Obsolete
132132
//We should also decide if we even need any of those two properties.
133133
/// <summary>
134134
/// 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
@@ -617,8 +617,11 @@ private IWriteOperation<BsonDocument> CreateCreateCollectionOperation<TDocument>
617617
new(documentSerializer, serializerRegistry, translationOptions: translationOptions) :
618618
new(documentSerializer, options.SerializationDomain, translationOptions: translationOptions);
619619

620+
//var serializationDomain = Settings.SerializationDomain.WithSerializerRegistry(serializerRegistry); //FP This could be the way to go
621+
620622
//DOMAIN-API This will need to go away
621-
var clusteredIndex = options.SerializationDomain is null? options.ClusteredIndex?.Render(documentSerializer, serializerRegistry, translationOptions):
623+
var clusteredIndex = options.SerializationDomain is null?
624+
options.ClusteredIndex?.Render(documentSerializer, serializerRegistry, translationOptions):
622625
options.ClusteredIndex?.Render(documentSerializer, options.SerializationDomain, translationOptions);
623626

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

0 commit comments

Comments
 (0)