Add CodecRegistries#withUuidRepresentation factory method (#864)

jyemin · web-flow · commit 92f064dfc6dc · 2022-02-03T18:03:52.000-05:00
JAVA-4461
diff --git a/bson/src/main/org/bson/Document.java b/bson/src/main/org/bson/Document.java
@@ -29,7 +29,6 @@
 import org.bson.codecs.ValueCodecProvider;
 import org.bson.codecs.configuration.CodecRegistry;
 import org.bson.conversions.Bson;
-import org.bson.internal.CodecRegistryHelper;
 import org.bson.json.JsonMode;
 import org.bson.json.JsonReader;
 import org.bson.json.JsonWriter;
@@ -51,6 +50,7 @@
 import static org.bson.assertions.Assertions.isTrue;
 import static org.bson.assertions.Assertions.notNull;
 import static org.bson.codecs.configuration.CodecRegistries.fromProviders;
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation;
 
 /**
  * A representation of a document as a {@code Map}.  All iterators will traverse the elements in insertion order, as with {@code
@@ -61,10 +61,8 @@
  */
 public class Document implements Map<String, Object>, Serializable, Bson {
     private static final Codec<Document> DEFAULT_CODEC =
-            CodecRegistryHelper.createRegistry(
-                    fromProviders(asList(new ValueCodecProvider(), new IterableCodecProvider(),
-                            new BsonValueCodecProvider(), new DocumentCodecProvider(), new MapCodecProvider())),
-                    UuidRepresentation.STANDARD)
+            withUuidRepresentation(fromProviders(asList(new ValueCodecProvider(), new IterableCodecProvider(),
+                    new BsonValueCodecProvider(), new DocumentCodecProvider(), new MapCodecProvider())), UuidRepresentation.STANDARD)
                     .get(Document.class);
 
     private static final long serialVersionUID = 6297731997167536582L;
diff --git a/bson/src/main/org/bson/codecs/DocumentCodec.java b/bson/src/main/org/bson/codecs/DocumentCodec.java
@@ -115,6 +115,9 @@ public Object transform(final Object value) {
 
     @Override
     public Codec<Document> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {
+        if (this.uuidRepresentation.equals(uuidRepresentation)) {
+            return this;
+        }
         return new DocumentCodec(registry, bsonTypeCodecMap, idGenerator, valueTransformer, uuidRepresentation);
     }
 
diff --git a/bson/src/main/org/bson/codecs/IterableCodec.java b/bson/src/main/org/bson/codecs/IterableCodec.java
@@ -80,6 +80,9 @@ public Object transform(final Object objectToTransform) {
 
     @Override
     public Codec<Iterable> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {
+        if (this.uuidRepresentation.equals(uuidRepresentation)) {
+            return this;
+        }
         return new IterableCodec(registry, bsonTypeCodecMap, valueTransformer, uuidRepresentation);
     }
 
diff --git a/bson/src/main/org/bson/codecs/MapCodec.java b/bson/src/main/org/bson/codecs/MapCodec.java
@@ -101,6 +101,9 @@ public Object transform(final Object value) {
 
     @Override
     public Codec<Map<String, Object>> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {
+        if (this.uuidRepresentation.equals(uuidRepresentation)) {
+            return this;
+        }
         return new MapCodec(registry, bsonTypeCodecMap, valueTransformer, uuidRepresentation);
     }
 
diff --git a/bson/src/main/org/bson/codecs/OverridableUuidRepresentationUuidCodec.java b/bson/src/main/org/bson/codecs/OverridableUuidRepresentationUuidCodec.java
@@ -45,6 +45,9 @@ public OverridableUuidRepresentationUuidCodec(final UuidRepresentation uuidRepre
 
     @Override
     public Codec<UUID> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {
+        if (getUuidRepresentation().equals(uuidRepresentation)) {
+            return this;
+        }
         return new OverridableUuidRepresentationUuidCodec(uuidRepresentation);
     }
 }
diff --git a/bson/src/main/org/bson/codecs/configuration/CodecRegistries.java b/bson/src/main/org/bson/codecs/configuration/CodecRegistries.java
@@ -16,7 +16,9 @@
 
 package org.bson.codecs.configuration;
 
+import org.bson.UuidRepresentation;
 import org.bson.codecs.Codec;
+import org.bson.internal.OverridableUuidRepresentationCodecRegistry;
 import org.bson.internal.ProvidersCodecRegistry;
 
 import java.util.List;
@@ -30,6 +32,28 @@
  */
 public final class CodecRegistries {
 
+    /**
+     * Apply given {@link UuidRepresentation} to the given {@link CodecRegistry}.
+     *
+     * @param codecRegistry the code registry
+     * @param uuidRepresentation the uuid representation
+     * @since 4.5
+     */
+    public static CodecRegistry withUuidRepresentation(final CodecRegistry codecRegistry, final UuidRepresentation uuidRepresentation) {
+        if (codecRegistry instanceof OverridableUuidRepresentationCodecRegistry) {
+            OverridableUuidRepresentationCodecRegistry overridableUuidRepresentationCodecRegistry =
+                    (OverridableUuidRepresentationCodecRegistry) codecRegistry;
+            if (overridableUuidRepresentationCodecRegistry.getUuidRepresentation().equals(uuidRepresentation)) {
+                return codecRegistry;
+            } else {
+                return new OverridableUuidRepresentationCodecRegistry(overridableUuidRepresentationCodecRegistry.getWrapped(),
+                        uuidRepresentation);
+            }
+        } else {
+            return new OverridableUuidRepresentationCodecRegistry(codecRegistry, uuidRepresentation);
+        }
+    }
+
     /**
      * Creates a {@code CodecRegistry} from the provided list of {@code Codec} instances.
      *
diff --git a/bson/src/main/org/bson/internal/CodecRegistryHelper.java b/bson/src/main/org/bson/internal/CodecRegistryHelper.java
diff --git a/bson/src/main/org/bson/internal/OverridableUuidRepresentationCodecRegistry.java b/bson/src/main/org/bson/internal/OverridableUuidRepresentationCodecRegistry.java
@@ -30,7 +30,7 @@ public class OverridableUuidRepresentationCodecRegistry implements CycleDetectin
     private final CodecCache codecCache = new CodecCache();
     private final UuidRepresentation uuidRepresentation;
 
-    OverridableUuidRepresentationCodecRegistry(final CodecProvider wrapped, final UuidRepresentation uuidRepresentation) {
+    public OverridableUuidRepresentationCodecRegistry(final CodecProvider wrapped, final UuidRepresentation uuidRepresentation) {
         this.uuidRepresentation = notNull("uuidRepresentation", uuidRepresentation);
         this.wrapped = notNull("wrapped", wrapped);
     }
diff --git a/bson/src/test/unit/org/bson/codecs/configuration/CodeRegistriesSpecification.groovy b/bson/src/test/unit/org/bson/codecs/configuration/CodeRegistriesSpecification.groovy
@@ -23,12 +23,16 @@ import org.bson.codecs.IntegerCodec
 import org.bson.codecs.LongCodec
 import org.bson.codecs.UuidCodec
 import org.bson.codecs.ValueCodecProvider
+import org.bson.internal.OverridableUuidRepresentationCodecRegistry
 import org.bson.internal.ProvidersCodecRegistry
 import spock.lang.Specification
 
 import static CodecRegistries.fromCodecs
 import static CodecRegistries.fromProviders
 import static CodecRegistries.fromRegistries
+import static org.bson.UuidRepresentation.STANDARD
+import static org.bson.UuidRepresentation.UNSPECIFIED
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation
 
 class CodeRegistriesSpecification extends Specification {
     def 'fromCodec should return a SingleCodecRegistry'() {
@@ -70,4 +74,31 @@ class CodeRegistriesSpecification extends Specification {
         registry.get(UUID).is(uuidCodec)
         registry.get(Integer) instanceof IntegerCodec
     }
+
+    def 'withUuidRepresentation should apply uuid representation'() {
+        given:
+        def registry = fromProviders(new ValueCodecProvider());
+
+        when:
+        def registryWithStandard = withUuidRepresentation(registry, STANDARD)
+
+        then:
+        registryWithStandard instanceof OverridableUuidRepresentationCodecRegistry
+        (registryWithStandard as OverridableUuidRepresentationCodecRegistry).uuidRepresentation == STANDARD
+        (registryWithStandard as OverridableUuidRepresentationCodecRegistry).wrapped == registry
+
+        when:
+        def registryWithUnspecified = withUuidRepresentation(registryWithStandard, UNSPECIFIED)
+
+        then:
+        registryWithUnspecified instanceof OverridableUuidRepresentationCodecRegistry
+        (registryWithUnspecified as OverridableUuidRepresentationCodecRegistry).uuidRepresentation == UNSPECIFIED
+        (registryWithUnspecified as OverridableUuidRepresentationCodecRegistry).wrapped == registry
+
+        when:
+        def registryWithUnspecifiedTwo = withUuidRepresentation(registryWithUnspecified, UNSPECIFIED)
+
+        then:
+        registryWithUnspecifiedTwo === registryWithUnspecified
+    }
 }
diff --git a/driver-core/src/main/com/mongodb/BasicDBObject.java b/driver-core/src/main/com/mongodb/BasicDBObject.java
@@ -29,7 +29,6 @@
 import org.bson.codecs.EncoderContext;
 import org.bson.codecs.configuration.CodecRegistry;
 import org.bson.conversions.Bson;
-import org.bson.internal.CodecRegistryHelper;
 import org.bson.io.BasicOutputBuffer;
 import org.bson.io.OutputBuffer;
 import org.bson.json.JsonMode;
@@ -46,6 +45,8 @@
 import java.util.Map;
 import java.util.TreeSet;
 
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation;
+
 /**
  * A basic implementation of BSON object that is MongoDB specific. A {@code DBObject} can be created as follows, using this class:
  * <pre>
@@ -60,9 +61,7 @@ public class BasicDBObject extends BasicBSONObject implements DBObject, Bson {
     private static final long serialVersionUID = -4415279469780082174L;
 
     private static final Codec<BasicDBObject> DEFAULT_CODEC =
-            CodecRegistryHelper.createRegistry(
-                    DBObjectCodec.getDefaultRegistry(),
-                    UuidRepresentation.STANDARD)
+            withUuidRepresentation(DBObjectCodec.getDefaultRegistry(), UuidRepresentation.STANDARD)
                     .get(BasicDBObject.class);
 
     /**
diff --git a/driver-core/src/main/com/mongodb/DBObjectCodec.java b/driver-core/src/main/com/mongodb/DBObjectCodec.java
@@ -207,6 +207,9 @@ public DBObject generateIdIfAbsentFromDocument(final DBObject document) {
 
     @Override
     public Codec<DBObject> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {
+        if (this.uuidRepresentation.equals(uuidRepresentation)) {
+            return this;
+        }
         return new DBObjectCodec(codecRegistry, bsonTypeCodecMap, objectFactory, uuidRepresentation);
     }
 
diff --git a/driver-reactive-streams/src/main/com/mongodb/reactivestreams/client/internal/MongoClientImpl.java b/driver-reactive-streams/src/main/com/mongodb/reactivestreams/client/internal/MongoClientImpl.java
@@ -47,7 +47,7 @@
 import java.util.concurrent.atomic.AtomicBoolean;
 
 import static com.mongodb.assertions.Assertions.notNull;
-import static org.bson.internal.CodecRegistryHelper.createRegistry;
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation;
 
 
 /**
@@ -92,8 +92,8 @@ private MongoClientImpl(final MongoClientSettings settings, final Cluster cluste
         }
         this.externalResourceCloser = externalResourceCloser;
         this.mongoOperationPublisher = new MongoOperationPublisher<>(Document.class,
-                                                                     createRegistry(settings.getCodecRegistry(),
-                                                                                    settings.getUuidRepresentation()),
+                                                                     withUuidRepresentation(settings.getCodecRegistry(),
+                                                                     settings.getUuidRepresentation()),
                                                                      settings.getReadPreference(),
                                                                      settings.getReadConcern(), settings.getWriteConcern(),
                                                                      settings.getRetryWrites(), settings.getRetryReads(),
diff --git a/driver-reactive-streams/src/main/com/mongodb/reactivestreams/client/internal/MongoOperationPublisher.java b/driver-reactive-streams/src/main/com/mongodb/reactivestreams/client/internal/MongoOperationPublisher.java
@@ -84,7 +84,7 @@
 import static com.mongodb.assertions.Assertions.notNull;
 import static java.util.Collections.singletonList;
 import static java.util.stream.Collectors.toList;
-import static org.bson.internal.CodecRegistryHelper.createRegistry;
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation;
 
 public final class MongoOperationPublisher<T> {
 
@@ -179,7 +179,7 @@ <D> MongoOperationPublisher<D> withNamespaceAndDocumentClass(final MongoNamespac
 
     MongoOperationPublisher<T> withCodecRegistry(final CodecRegistry codecRegistry) {
         return new MongoOperationPublisher<>(getNamespace(), getDocumentClass(),
-                                             createRegistry(notNull("codecRegistry", codecRegistry), uuidRepresentation),
+                                             withUuidRepresentation(notNull("codecRegistry", codecRegistry), uuidRepresentation),
                                              getReadPreference(), getReadConcern(), getWriteConcern(), getRetryWrites(), getRetryReads(),
                                              uuidRepresentation, executor);
     }
diff --git a/driver-sync/src/main/com/mongodb/client/internal/MongoClientImpl.java b/driver-sync/src/main/com/mongodb/client/internal/MongoClientImpl.java
@@ -53,7 +53,7 @@
 import static com.mongodb.assertions.Assertions.notNull;
 import static com.mongodb.client.internal.Crypts.createCrypt;
 import static com.mongodb.internal.event.EventListenerHelper.getCommandListener;
-import static org.bson.internal.CodecRegistryHelper.createRegistry;
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation;
 
 public final class MongoClientImpl implements MongoClient {
 
@@ -76,7 +76,7 @@ public MongoClientImpl(final Cluster cluster, final MongoDriverInformation mongo
                     + SynchronousContextProvider.class.getName() + " when using the synchronous driver");
         }
         this.delegate = new MongoClientDelegate(notNull("cluster", cluster),
-                createRegistry(settings.getCodecRegistry(), settings.getUuidRepresentation()), this, operationExecutor,
+                withUuidRepresentation(settings.getCodecRegistry(), settings.getUuidRepresentation()), this, operationExecutor,
                 autoEncryptionSettings == null ? null : createCrypt(this, autoEncryptionSettings), settings.getServerApi(),
                 (SynchronousContextProvider) settings.getContextProvider());
     }
diff --git a/driver-sync/src/main/com/mongodb/client/internal/MongoCollectionImpl.java b/driver-sync/src/main/com/mongodb/client/internal/MongoCollectionImpl.java
@@ -81,7 +81,7 @@
 import static com.mongodb.internal.bulk.WriteRequest.Type.REPLACE;
 import static com.mongodb.internal.bulk.WriteRequest.Type.UPDATE;
 import static java.util.Collections.singletonList;
-import static org.bson.internal.CodecRegistryHelper.createRegistry;
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation;
 
 class MongoCollectionImpl<TDocument> implements MongoCollection<TDocument> {
     private final MongoNamespace namespace;
@@ -152,7 +152,7 @@ public <NewTDocument> MongoCollection<NewTDocument> withDocumentClass(final Clas
 
     @Override
     public MongoCollection<TDocument> withCodecRegistry(final CodecRegistry codecRegistry) {
-        return new MongoCollectionImpl<>(namespace, documentClass, createRegistry(codecRegistry, uuidRepresentation),
+        return new MongoCollectionImpl<>(namespace, documentClass, withUuidRepresentation(codecRegistry, uuidRepresentation),
                 readPreference, writeConcern, retryWrites, retryReads, readConcern, uuidRepresentation, executor);
     }
 
diff --git a/driver-sync/src/main/com/mongodb/client/internal/MongoDatabaseImpl.java b/driver-sync/src/main/com/mongodb/client/internal/MongoDatabaseImpl.java
@@ -53,7 +53,7 @@
 
 import static com.mongodb.MongoNamespace.checkDatabaseNameValidity;
 import static com.mongodb.assertions.Assertions.notNull;
-import static org.bson.internal.CodecRegistryHelper.createRegistry;
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation;
 
 /**
  * This class is not part of the public API and may be removed or changed at any time.
@@ -111,7 +111,7 @@ public ReadConcern getReadConcern() {
 
     @Override
     public MongoDatabase withCodecRegistry(final CodecRegistry codecRegistry) {
-        return new MongoDatabaseImpl(name, createRegistry(codecRegistry, uuidRepresentation), readPreference, writeConcern, retryWrites,
+        return new MongoDatabaseImpl(name, withUuidRepresentation(codecRegistry, uuidRepresentation), readPreference, writeConcern, retryWrites,
                 retryReads, readConcern, uuidRepresentation, executor);
     }
 
diff --git a/driver-sync/src/test/unit/com/mongodb/client/MongoClientSpecification.groovy b/driver-sync/src/test/unit/com/mongodb/client/MongoClientSpecification.groovy
@@ -50,6 +50,7 @@ import static com.mongodb.client.internal.TestHelper.execute
 import static org.bson.UuidRepresentation.STANDARD
 import static org.bson.UuidRepresentation.UNSPECIFIED
 import static org.bson.codecs.configuration.CodecRegistries.fromProviders
+import static org.bson.codecs.configuration.CodecRegistries.withUuidRepresentation
 import static spock.util.matcher.HamcrestSupport.expect
 
 class MongoClientSpecification extends Specification {
@@ -74,7 +75,7 @@ class MongoClientSpecification extends Specification {
         expect database, isTheSameAs(expectedDatabase)
 
         where:
-        expectedDatabase << new MongoDatabaseImpl('name', codecRegistry, secondary(),
+        expectedDatabase << new MongoDatabaseImpl('name', withUuidRepresentation(codecRegistry, UNSPECIFIED), secondary(),
                 WriteConcern.MAJORITY, true, true, ReadConcern.MAJORITY, UNSPECIFIED, new TestOperationExecutor([]))
     }
 
@@ -90,22 +91,22 @@ class MongoClientSpecification extends Specification {
 
         then:
         expect listDatabasesIterable, isTheSameAs(new ListDatabasesIterableImpl<>(session, Document,
-                getDefaultCodecRegistry(), primary(), executor, true))
+                withUuidRepresentation(getDefaultCodecRegistry(), UNSPECIFIED), primary(), executor, true))
 
         when:
         listDatabasesIterable = execute(listDatabasesMethod, session, BsonDocument)
 
         then:
         expect listDatabasesIterable, isTheSameAs(new ListDatabasesIterableImpl<>(session, BsonDocument,
-                getDefaultCodecRegistry(), primary(), executor, true))
+                withUuidRepresentation(getDefaultCodecRegistry(), UNSPECIFIED), primary(), executor, true))
 
         when:
         def listDatabaseNamesIterable = execute(listDatabasesNamesMethod, session) as MongoIterable<String>
 
         then:
         // listDatabaseNamesIterable is an instance of a MappingIterable, so have to get the mapped iterable inside it
         expect listDatabaseNamesIterable.getMapped(), isTheSameAs(new ListDatabasesIterableImpl<>(session, BsonDocument,
-                getDefaultCodecRegistry(), primary(), executor, true).nameOnly(true))
+                withUuidRepresentation(getDefaultCodecRegistry(), UNSPECIFIED), primary(), executor, true).nameOnly(true))
 
         cleanup:
         client?.close()
diff --git a/driver-sync/src/test/unit/com/mongodb/client/internal/MongoCollectionSpecification.groovy b/driver-sync/src/test/unit/com/mongodb/client/internal/MongoCollectionSpecification.groovy
@@ -107,7 +107,6 @@ import static com.mongodb.internal.bulk.WriteRequest.Type.UPDATE
 import static java.util.concurrent.TimeUnit.MILLISECONDS
 import static org.bson.UuidRepresentation.JAVA_LEGACY
 import static org.bson.UuidRepresentation.STANDARD
-import static org.bson.UuidRepresentation.UNSPECIFIED
 import static org.bson.codecs.configuration.CodecRegistries.fromProviders
 import static spock.util.matcher.HamcrestSupport.expect
 
@@ -151,15 +150,6 @@ class MongoCollectionSpecification extends Specification {
 
         when:
         def collection = new MongoCollectionImpl(namespace, Document, codecRegistry, readPreference, ACKNOWLEDGED,
-                true, true, readConcern, UNSPECIFIED, executor).withCodecRegistry(newCodecRegistry)
-
-        then:
-        collection.getCodecRegistry() == newCodecRegistry
-        expect collection, isTheSameAs(new MongoCollectionImpl(namespace, Document, newCodecRegistry, readPreference, ACKNOWLEDGED,
-                true, true, readConcern, UNSPECIFIED, executor))
-
-        when:
-        collection = new MongoCollectionImpl(namespace, Document, codecRegistry, readPreference, ACKNOWLEDGED,
                 true, true, readConcern, STANDARD, executor).withCodecRegistry(newCodecRegistry)
 
         then:
diff --git a/driver-sync/src/test/unit/com/mongodb/client/internal/MongoDatabaseSpecification.groovy b/driver-sync/src/test/unit/com/mongodb/client/internal/MongoDatabaseSpecification.groovy

Original file line number	Diff line number	Diff line change
`@@ -115,6 +115,9 @@ public Object transform(final Object value) {`
`115`	`115`
`116`	`116`	`@Override`
`117`	`117`	`public Codec<Document> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {`
	`118`	`+ if (this.uuidRepresentation.equals(uuidRepresentation)) {`
	`119`	`+ return this;`
	`120`	`+ }`
`118`	`121`	`return new DocumentCodec(registry, bsonTypeCodecMap, idGenerator, valueTransformer, uuidRepresentation);`
`119`	`122`	`}`
`120`	`123`
Original file line number	Diff line number	Diff line change
`@@ -80,6 +80,9 @@ public Object transform(final Object objectToTransform) {`
`80`	`80`
`81`	`81`	`@Override`
`82`	`82`	`public Codec<Iterable> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {`
	`83`	`+ if (this.uuidRepresentation.equals(uuidRepresentation)) {`
	`84`	`+ return this;`
	`85`	`+ }`
`83`	`86`	`return new IterableCodec(registry, bsonTypeCodecMap, valueTransformer, uuidRepresentation);`
`84`	`87`	`}`
`85`	`88`
Original file line number	Diff line number	Diff line change
`@@ -101,6 +101,9 @@ public Object transform(final Object value) {`
`101`	`101`
`102`	`102`	`@Override`
`103`	`103`	`public Codec<Map<String, Object>> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {`
	`104`	`+ if (this.uuidRepresentation.equals(uuidRepresentation)) {`
	`105`	`+ return this;`
	`106`	`+ }`
`104`	`107`	`return new MapCodec(registry, bsonTypeCodecMap, valueTransformer, uuidRepresentation);`
`105`	`108`	`}`
`106`	`109`
Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,9 @@ public OverridableUuidRepresentationUuidCodec(final UuidRepresentation uuidRepre`
`45`	`45`
`46`	`46`	`@Override`
`47`	`47`	`public Codec<UUID> withUuidRepresentation(final UuidRepresentation uuidRepresentation) {`
	`48`	`+ if (getUuidRepresentation().equals(uuidRepresentation)) {`
	`49`	`+ return this;`
	`50`	`+ }`
`48`	`51`	`return new OverridableUuidRepresentationUuidCodec(uuidRepresentation);`
`49`	`52`	`}`
`50`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public class OverridableUuidRepresentationCodecRegistry implements CycleDetectin`
`30`	`30`	`private final CodecCache codecCache = new CodecCache();`
`31`	`31`	`private final UuidRepresentation uuidRepresentation;`
`32`	`32`
`33`		`- OverridableUuidRepresentationCodecRegistry(final CodecProvider wrapped, final UuidRepresentation uuidRepresentation) {`
	`33`	`+ public OverridableUuidRepresentationCodecRegistry(final CodecProvider wrapped, final UuidRepresentation uuidRepresentation) {`
`34`	`34`	`this.uuidRepresentation = notNull("uuidRepresentation", uuidRepresentation);`
`35`	`35`	`this.wrapped = notNull("wrapped", wrapped);`
`36`	`36`	`}`