vkagamlyk commented on code in PR #2576:
URL: https://github.com/apache/tinkerpop/pull/2576#discussion_r1583698290
##########
gremlin-driver/src/test/java/org/apache/tinkerpop/gremlin/driver/ClusterConfigTest.java:
##########
@@ -33,40 +32,21 @@
public class ClusterConfigTest {
@Test
- public void shouldPropagateSerializerConstraintsForGraphSON3() {
+ public void shouldPropagateSerializerConstraintsForGraphSON4() {
final Configuration config = new BaseConfiguration();
config.setProperty("serializer.config.maxNumberLength", 999);
config.setProperty("serializer.config.maxStringLength", 123456);
config.setProperty("serializer.config.maxNestingDepth", 55);
config.setProperty("hosts", Arrays.asList("localhost"));
- config.setProperty("serializer.className",
GraphSONMessageSerializerV3.class.getCanonicalName());
+ config.setProperty("serializer.className",
GraphSONMessageSerializerV4.class.getCanonicalName());
final Cluster cluster = Cluster.open(config);
- assertTrue(cluster.getSerializer() instanceof
GraphSONMessageSerializerV3);
- final GraphSONMessageSerializerV3 serV3 =
(GraphSONMessageSerializerV3) cluster.getSerializer();
+ assertTrue(cluster.getSerializer() instanceof
GraphSONMessageSerializerV4);
+ final GraphSONMessageSerializerV4 serV3 =
(GraphSONMessageSerializerV4) cluster.getSerializer();
Review Comment:
nit: var name
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
