You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2017/06/09 18:50:36 UTC
[1/5] tinkerpop git commit: TINKERPOP-929 Remove deprecated fields
from TinkerGraph.
Repository: tinkerpop
Updated Branches:
refs/heads/TINKERPOP-929 [created] 578a93cfc
TINKERPOP-929 Remove deprecated fields from TinkerGraph.
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/4d0a5fb3
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/4d0a5fb3
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/4d0a5fb3
Branch: refs/heads/TINKERPOP-929
Commit: 4d0a5fb3e070ac4db066537f8f89c10c021e731e
Parents: 512ca3a
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Fri Jun 9 10:06:26 2017 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Fri Jun 9 10:06:26 2017 -0400
----------------------------------------------------------------------
CHANGELOG.asciidoc | 1 +
docs/src/upgrade/release-3.3.x.asciidoc | 1 +
.../tinkergraph/structure/TinkerGraph.java | 31 --------------------
3 files changed, 2 insertions(+), 31 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/4d0a5fb3/CHANGELOG.asciidoc
----------------------------------------------------------------------
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index b445a00..aa3ca46 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -32,6 +32,7 @@ TinkerPop 3.3.0 (Release Date: NOT OFFICIALLY RELEASED YET)
* Added "attachment requisite" `VertexProperty.element()` and `Property.element()` data in GraphSON serialization.
* GraphSON 3.0 is now the default serialization format in TinkerGraph and Gremlin Server.
* Established the GraphSON 3.0 format.
+* Removed previously deprecated TinkerGraph configuration member variables.
* Removed previously deprecated `OpSelectorHandler.errorMeter` and `AbstractEvalOpProcessor.errorMeter` fields.
* Removed previously deprecated `AbstractEvalOpProcessor.validBindingName` field.
* Removed previously deprecated `SimpleAuthenticator.CONFIG_CREDENTIALS_LOCATION` field.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/4d0a5fb3/docs/src/upgrade/release-3.3.x.asciidoc
----------------------------------------------------------------------
diff --git a/docs/src/upgrade/release-3.3.x.asciidoc b/docs/src/upgrade/release-3.3.x.asciidoc
index dbb40b4..800fd7a 100644
--- a/docs/src/upgrade/release-3.3.x.asciidoc
+++ b/docs/src/upgrade/release-3.3.x.asciidoc
@@ -244,6 +244,7 @@ The following deprecated classes, methods or fields have been removed in this ve
** `org.apache.tinkerpop.gremlin.spark.groovy.plugin.SparkGremlinPlugin`
* `tinkergraph-gremlin`
** `org.apache.tinkerpop.gremlin.tinkergraph.groovy.plugin.TinkerGraphGremlinPlugin`
+** `org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph.CONFIG_*`
Please see the javadoc deprecation notes or upgrade documentation specific to when the deprecation took place to
understand how to resolve this breaking change.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/4d0a5fb3/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java b/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java
index 79503ea..770cd6d 100644
--- a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java
+++ b/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java
@@ -75,37 +75,6 @@ public final class TinkerGraph implements Graph {
this.setProperty(Graph.GRAPH, TinkerGraph.class.getName());
}};
- /**
- * @deprecated As of release 3.1.0, replaced by {@link TinkerGraph#GREMLIN_TINKERGRAPH_VERTEX_ID_MANAGER}
- */
- @Deprecated
- public static final String CONFIG_VERTEX_ID = "gremlin.tinkergraph.vertexIdManager";
- /**
- * @deprecated As of release 3.1.0, replaced by {@link TinkerGraph#GREMLIN_TINKERGRAPH_EDGE_ID_MANAGER}
- */
- @Deprecated
- public static final String CONFIG_EDGE_ID = "gremlin.tinkergraph.edgeIdManager";
- /**
- * @deprecated As of release 3.1.0, replaced by {@link TinkerGraph#GREMLIN_TINKERGRAPH_VERTEX_PROPERTY_ID_MANAGER}
- */
- @Deprecated
- public static final String CONFIG_VERTEX_PROPERTY_ID = "gremlin.tinkergraph.vertexPropertyIdManager";
- /**
- * @deprecated As of release 3.1.0, replaced by {@link TinkerGraph#GREMLIN_TINKERGRAPH_DEFAULT_VERTEX_PROPERTY_CARDINALITY}
- */
- @Deprecated
- public static final String CONFIG_DEFAULT_VERTEX_PROPERTY_CARDINALITY = "gremlin.tinkergraph.defaultVertexPropertyCardinality";
- /**
- * @deprecated As of release 3.1.0, replaced by {@link TinkerGraph#GREMLIN_TINKERGRAPH_GRAPH_LOCATION}
- */
- @Deprecated
- public static final String CONFIG_GRAPH_LOCATION = "gremlin.tinkergraph.graphLocation";
- /**
- * @deprecated As of release 3.1.0, replaced by {@link TinkerGraph#GREMLIN_TINKERGRAPH_GRAPH_FORMAT}
- */
- @Deprecated
- public static final String CONFIG_GRAPH_FORMAT = "gremlin.tinkergraph.graphFormat";
- ///////
public static final String GREMLIN_TINKERGRAPH_VERTEX_ID_MANAGER = "gremlin.tinkergraph.vertexIdManager";
public static final String GREMLIN_TINKERGRAPH_EDGE_ID_MANAGER = "gremlin.tinkergraph.edgeIdManager";
public static final String GREMLIN_TINKERGRAPH_VERTEX_PROPERTY_ID_MANAGER = "gremlin.tinkergraph.vertexPropertyIdManager";
[5/5] tinkerpop git commit: TINKERPOP-1408 Removed deprecated
Io.Builder#registry()
Posted by sp...@apache.org.
TINKERPOP-1408 Removed deprecated Io.Builder#registry()
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/578a93cf
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/578a93cf
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/578a93cf
Branch: refs/heads/TINKERPOP-929
Commit: 578a93cfcc330acf2e3f766a64934e3b3eb545af
Parents: 89653bb
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Fri Jun 9 11:52:07 2017 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Fri Jun 9 11:52:07 2017 -0400
----------------------------------------------------------------------
CHANGELOG.asciidoc | 1 +
docs/src/upgrade/release-3.3.x.asciidoc | 1 +
.../apache/tinkerpop/gremlin/structure/io/Io.java | 15 +--------------
.../gremlin/structure/io/graphml/GraphMLIo.java | 10 ----------
.../gremlin/structure/io/graphson/GraphSONIo.java | 16 +---------------
.../tinkerpop/gremlin/structure/io/gryo/GryoIo.java | 16 +---------------
.../tinkergraph/structure/TinkerGraphTest.java | 12 +-----------
7 files changed, 6 insertions(+), 65 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/578a93cf/CHANGELOG.asciidoc
----------------------------------------------------------------------
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index ad08fe1..e43531f 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -32,6 +32,7 @@ TinkerPop 3.3.0 (Release Date: NOT OFFICIALLY RELEASED YET)
* Added "attachment requisite" `VertexProperty.element()` and `Property.element()` data in GraphSON serialization.
* GraphSON 3.0 is now the default serialization format in TinkerGraph and Gremlin Server.
* Established the GraphSON 3.0 format.
+* Removed previously deprecated `Io.Builder#registry(IoRegistry)` method.
* Removed previously deprecated `GryoMessageSerializerV1d0(GryoMapper)` constructor.
* Removed previously deprecated `TinkerIoRegistry`.
* Removed previously deprecated `getInstance()` methods on all TinkerPop classes.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/578a93cf/docs/src/upgrade/release-3.3.x.asciidoc
----------------------------------------------------------------------
diff --git a/docs/src/upgrade/release-3.3.x.asciidoc b/docs/src/upgrade/release-3.3.x.asciidoc
index 85a4f53..b3617e7 100644
--- a/docs/src/upgrade/release-3.3.x.asciidoc
+++ b/docs/src/upgrade/release-3.3.x.asciidoc
@@ -199,6 +199,7 @@ The following deprecated classes, methods or fields have been removed in this ve
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_PROCESS_COMPUTER`
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_ENVIRONMENT`
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_ENVIRONMENT_INTEGRATE`
+** `org.apache.tinkerpop.gremlin.structure.io.Io.Builder#registry(IoRegistry)`
* `gremlin-driver`
** `org.apache.tinkerpop.gremlin.driver.ser.GryoMessageSerializerV1d0(GryoMapper)`
* `gremlin-groovy`
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/578a93cf/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Io.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Io.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Io.java
index 820f474..0a04979 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Io.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Io.java
@@ -85,23 +85,10 @@ public interface Io<R extends GraphReader.ReaderBuilder, W extends GraphWriter.W
* that class will be passed to {@link Graph#io(Io.Builder)} by the user.
*/
public interface Builder<I extends Io> {
-
- /**
- * Providers use this method to supply an {@link IoRegistry} to the {@link Io} implementation. End-users
- * should not use this method directly. If a user wants to register custom serializers, then such things
- * can be done via calls to {@link Io#mapper()} after the {@link Io} is constructed via
- * {@link Graph#io(Io.Builder)}.
- * @deprecated As of release 3.2.2, replaced by {@link #onMapper(Consumer)}.
- */
- @Deprecated
- public Builder<? extends Io> registry(final IoRegistry registry);
-
/**
* Allows a {@link Graph} implementation to have full control over the {@link Mapper.Builder} instance.
* Typically, the implementation will just pass in its {@link IoRegistry} implementation so that the
- * {@link Mapper} that gets built will have knowledge of any custom classes and serializers it may have. Note
- * that if {@link #registry(IoRegistry)} is also set on a {@code Builder} instance it will be applied first
- * prior to that instance being passed to {@code Consumer}.
+ * {@link Mapper} that gets built will have knowledge of any custom classes and serializers it may have.
* <p/>
* End-users should not use this method directly. If a user wants to register custom serializers, then such
* things can be done via calls to {@link Io#mapper()} after the {@link Io} is constructed via
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/578a93cf/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphml/GraphMLIo.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphml/GraphMLIo.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphml/GraphMLIo.java
index f595fbe..8f3dab3 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphml/GraphMLIo.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphml/GraphMLIo.java
@@ -102,16 +102,6 @@ public final class GraphMLIo implements Io<GraphMLReader.Builder, GraphMLWriter.
private Graph graph;
private Consumer<Mapper.Builder> onMapper = null;
- /**
- * @deprecated As of release 3.2.2, replaced by {@link #onMapper(Consumer)}.
- */
- @Deprecated
- @Override
- public Io.Builder<GraphMLIo> registry(final IoRegistry registry) {
- // GraphML doesn't make use of a registry but the contract should simply exist
- return this;
- }
-
@Override
public Io.Builder<? extends Io> onMapper(final Consumer<Mapper.Builder> onMapper) {
this.onMapper = onMapper;
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/578a93cf/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONIo.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONIo.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONIo.java
index 0c92e1c..82ffdad 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONIo.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONIo.java
@@ -38,13 +38,11 @@ import java.util.function.Consumer;
* @author Stephen Mallette (http://stephen.genoprime.com)
*/
public final class GraphSONIo implements Io<GraphSONReader.Builder, GraphSONWriter.Builder, GraphSONMapper.Builder> {
- private final IoRegistry registry;
private final Graph graph;
private final Optional<Consumer<Mapper.Builder>> onMapper;
private final GraphSONVersion version;
private GraphSONIo(final Builder builder) {
- this.registry = builder.registry;
this.graph = builder.graph;
this.onMapper = Optional.ofNullable(builder.onMapper);
this.version = builder.version;
@@ -71,8 +69,7 @@ public final class GraphSONIo implements Io<GraphSONReader.Builder, GraphSONWrit
*/
@Override
public GraphSONMapper.Builder mapper() {
- final GraphSONMapper.Builder builder = (null == this.registry) ?
- GraphSONMapper.build().version(version) : GraphSONMapper.build().version(version).addRegistry(this.registry);
+ final GraphSONMapper.Builder builder = GraphSONMapper.build().version(version);
onMapper.ifPresent(c -> c.accept(builder));
return builder;
}
@@ -113,7 +110,6 @@ public final class GraphSONIo implements Io<GraphSONReader.Builder, GraphSONWrit
public final static class Builder implements Io.Builder<GraphSONIo> {
- private IoRegistry registry = null;
private Graph graph;
private Consumer<Mapper.Builder> onMapper = null;
private final GraphSONVersion version;
@@ -122,16 +118,6 @@ public final class GraphSONIo implements Io<GraphSONReader.Builder, GraphSONWrit
this.version = version;
}
- /**
- * @deprecated As of release 3.2.2, replaced by {@link #onMapper(Consumer)}.
- */
- @Deprecated
- @Override
- public Io.Builder<GraphSONIo> registry(final IoRegistry registry) {
- this.registry = registry;
- return this;
- }
-
@Override
public Io.Builder<? extends Io> onMapper(final Consumer<Mapper.Builder> onMapper) {
this.onMapper = onMapper;
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/578a93cf/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoIo.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoIo.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoIo.java
index 86d96c9..9ce4cbe 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoIo.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoIo.java
@@ -39,12 +39,10 @@ import java.util.function.Consumer;
*/
public final class GryoIo implements Io<GryoReader.Builder, GryoWriter.Builder, GryoMapper.Builder> {
- private final IoRegistry registry;
private final Graph graph;
private Optional<Consumer<Mapper.Builder>> onMapper;
private GryoIo(final Builder builder) {
- this.registry = builder.registry;
this.graph = builder.graph;
this.onMapper = Optional.ofNullable(builder.onMapper);
}
@@ -69,7 +67,7 @@ public final class GryoIo implements Io<GryoReader.Builder, GryoWriter.Builder,
*/
@Override
public GryoMapper.Builder mapper() {
- final GryoMapper.Builder builder = (null == this.registry) ? GryoMapper.build() : GryoMapper.build().addRegistry(this.registry);
+ final GryoMapper.Builder builder = GryoMapper.build();
onMapper.ifPresent(c -> c.accept(builder));
return builder;
}
@@ -99,21 +97,9 @@ public final class GryoIo implements Io<GryoReader.Builder, GryoWriter.Builder,
}
public final static class Builder implements Io.Builder<GryoIo> {
-
- private IoRegistry registry = null;
private Graph graph;
private Consumer<Mapper.Builder> onMapper = null;
- /**
- * @deprecated As of release 3.2.2, replaced by {@link #onMapper(Consumer)}.
- */
- @Deprecated
- @Override
- public Io.Builder<GryoIo> registry(final IoRegistry registry) {
- this.registry = registry;
- return this;
- }
-
@Override
public Io.Builder<? extends Io> onMapper(final Consumer<Mapper.Builder> onMapper) {
this.onMapper = onMapper;
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/578a93cf/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
index a85b0ee..edf2127 100644
--- a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
+++ b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
@@ -32,7 +32,6 @@ import org.apache.tinkerpop.gremlin.structure.io.Io;
import org.apache.tinkerpop.gremlin.structure.io.GraphReader;
import org.apache.tinkerpop.gremlin.structure.io.GraphWriter;
import org.apache.tinkerpop.gremlin.structure.io.IoCore;
-import org.apache.tinkerpop.gremlin.structure.io.IoRegistry;
import org.apache.tinkerpop.gremlin.structure.io.IoTest;
import org.apache.tinkerpop.gremlin.structure.io.Mapper;
import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONReader;
@@ -518,7 +517,6 @@ public class TinkerGraphTest {
//Test write graph
graph.close();
- assertEquals(TestIoBuilder.calledRegistry, 0);
assertEquals(TestIoBuilder.calledOnMapper, 1);
assertEquals(TestIoBuilder.calledGraph, 1);
assertEquals(TestIoBuilder.calledCreate, 1);
@@ -531,7 +529,6 @@ public class TinkerGraphTest {
//Test read graph
final TinkerGraph readGraph = TinkerGraph.open(conf);
- assertEquals(TestIoBuilder.calledRegistry, 0);
assertEquals(TestIoBuilder.calledOnMapper, 1);
assertEquals(TestIoBuilder.calledGraph, 1);
assertEquals(TestIoBuilder.calledCreate, 1);
@@ -672,23 +669,16 @@ public class TinkerGraphTest {
public static class TestIoBuilder implements Io.Builder {
- static int calledRegistry, calledGraph, calledCreate, calledOnMapper;
+ static int calledGraph, calledCreate, calledOnMapper;
public TestIoBuilder(){
//Looks awkward to reset static vars inside a constructor, but makes sense from testing perspective
- calledRegistry = 0;
calledGraph = 0;
calledCreate = 0;
calledOnMapper = 0;
}
@Override
- public Io.Builder<? extends Io> registry(final IoRegistry registry) {
- calledRegistry++;
- return this;
- }
-
- @Override
public Io.Builder<? extends Io> onMapper(final Consumer onMapper) {
calledOnMapper++;
return this;
[3/5] tinkerpop git commit: TINKERPOP-1563 Removed deprecated
getInstance() methods
Posted by sp...@apache.org.
TINKERPOP-1563 Removed deprecated getInstance() methods
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/1a1ba583
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/1a1ba583
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/1a1ba583
Branch: refs/heads/TINKERPOP-929
Commit: 1a1ba58333d1ebae69d1e5e4d14e01255bb5bb40
Parents: 732c80b
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Fri Jun 9 10:58:57 2017 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Fri Jun 9 10:58:57 2017 -0400
----------------------------------------------------------------------
CHANGELOG.asciidoc | 2 +
docs/src/upgrade/release-3.3.x.asciidoc | 4 +
.../SingleGremlinScriptEngineManager.java | 7 -
.../structure/io/util/IoRegistryHelper.java | 3 +
.../structure/util/empty/EmptyGraph.java | 5 -
.../jsr223/SingleScriptEngineManagerTest.java | 1 -
.../server/GremlinResultSetIntegrateTest.java | 4 +-
gremlin-tools/gremlin-io-test/pom.xml | 4 +-
.../GraphSONUntypedCompatibilityTest.java | 4 +-
.../_3_2_3/manual-graphson-generator.groovy | 6 +-
.../_3_2_4/manual-graphson-generator.groovy | 6 +-
.../io/gryo/_3_2_3/manual-gryo-generator.groovy | 2 +-
.../io/gryo/_3_2_4/manual-gryo-generator.groovy | 2 +-
.../gremlin/hadoop/structure/HadoopGraph.java | 5 -
.../jsr223/TinkerGraphGremlinPlugin.java | 4 +-
.../tinkergraph/structure/TinkerIoRegistry.java | 272 -------------------
.../structure/TinkerIoRegistryV2d0.java | 8 -
.../TinkerGraphGryoSerializerTest.java | 80 ------
.../TinkerGraphGryoSerializerV1d0Test.java | 5 +-
19 files changed, 26 insertions(+), 398 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/CHANGELOG.asciidoc
----------------------------------------------------------------------
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index 404a570..1de020f 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -32,6 +32,8 @@ TinkerPop 3.3.0 (Release Date: NOT OFFICIALLY RELEASED YET)
* Added "attachment requisite" `VertexProperty.element()` and `Property.element()` data in GraphSON serialization.
* GraphSON 3.0 is now the default serialization format in TinkerGraph and Gremlin Server.
* Established the GraphSON 3.0 format.
+* Removed previously deprecated `TinkerIoRegistry`.
+* Removed previously deprecated `getInstance()` methods on all TinkerPop classes.
* Removed previously deprecated `VertexPropertyFeatures.supportsAddProperty()`.
* Removed previously deprecated TinkerGraph configuration member variables.
* Removed previously deprecated `OpSelectorHandler.errorMeter` and `AbstractEvalOpProcessor.errorMeter` fields.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/docs/src/upgrade/release-3.3.x.asciidoc
----------------------------------------------------------------------
diff --git a/docs/src/upgrade/release-3.3.x.asciidoc b/docs/src/upgrade/release-3.3.x.asciidoc
index 8d1c712..7d90e89 100644
--- a/docs/src/upgrade/release-3.3.x.asciidoc
+++ b/docs/src/upgrade/release-3.3.x.asciidoc
@@ -190,6 +190,7 @@ The following deprecated classes, methods or fields have been removed in this ve
* `gremlin-core`
** `org.apache.tinkerpop.gremlin.jsr223.CoreGremlinModule`
** `org.apache.tinkerpop.gremlin.jsr223.GremlinModule`
+** `org.apache.tinkerpop.gremlin.jsr223.SingleGremlinScriptEngineManager#getInstance()`
** `org.apache.tinkerpop.gremlin.jsr223.GremlinScriptEngineManager#addModule(GremlinModule)`
** `org.apache.tinkerpop.gremlin.jsr223.console.PluginAcceptor`
** `org.apache.tinkerpop.gremlin.structure.Graph.Features.VertexPropertyFeatures#supportsAddProperty()`
@@ -247,6 +248,9 @@ The following deprecated classes, methods or fields have been removed in this ve
* `tinkergraph-gremlin`
** `org.apache.tinkerpop.gremlin.tinkergraph.groovy.plugin.TinkerGraphGremlinPlugin`
** `org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph#CONFIG_*`
+** `org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistry`
+** `org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV1d0#getInstance()`
+** `org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV2d0#getInstance()`
Please see the javadoc deprecation notes or upgrade documentation specific to when the deprecation took place to
understand how to resolve this breaking change.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/SingleGremlinScriptEngineManager.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/SingleGremlinScriptEngineManager.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/SingleGremlinScriptEngineManager.java
index f9022dc..05cbe4c 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/SingleGremlinScriptEngineManager.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/SingleGremlinScriptEngineManager.java
@@ -28,13 +28,6 @@ public final class SingleGremlinScriptEngineManager {
private SingleGremlinScriptEngineManager() {}
- /**
- * @deprecated As of release 3.2.4, replaced by {@link #instance()}.
- */
- public static GremlinScriptEngineManager getInstance(){
- return instance();
- }
-
public static GremlinScriptEngineManager instance(){
return cached;
}
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/util/IoRegistryHelper.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/util/IoRegistryHelper.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/util/IoRegistryHelper.java
index 0688e6f..f878cfc 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/util/IoRegistryHelper.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/util/IoRegistryHelper.java
@@ -52,6 +52,9 @@ public final class IoRegistryHelper {
instanceMethod = clazz.getDeclaredMethod("instance"); // try for getInstance() ??
} catch (final NoSuchMethodException e) {
try {
+ // even though use of "getInstance" is no longer a thing in tinkerpop as of 3.3.0, perhaps
+ // others are using this style of naming. Doesn't seem to harm anything to continue to
+ // check for that method.
instanceMethod = clazz.getDeclaredMethod("getInstance"); // try for getInstance() ??
} catch (final NoSuchMethodException e2) {
// no instance() or getInstance() methods
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/empty/EmptyGraph.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/empty/EmptyGraph.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/empty/EmptyGraph.java
index 59464c5..fa10c99 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/empty/EmptyGraph.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/empty/EmptyGraph.java
@@ -215,11 +215,6 @@ public final class EmptyGraph implements Graph {
*/
public final class EmptyGraphVertexPropertyFeatures implements VertexPropertyFeatures {
@Override
- public boolean supportsAddProperty() {
- return false;
- }
-
- @Override
public boolean supportsRemoveProperty() {
return false;
}
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/jsr223/SingleScriptEngineManagerTest.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/jsr223/SingleScriptEngineManagerTest.java b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/jsr223/SingleScriptEngineManagerTest.java
index 6765c94..089f189 100644
--- a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/jsr223/SingleScriptEngineManagerTest.java
+++ b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/jsr223/SingleScriptEngineManagerTest.java
@@ -35,7 +35,6 @@ public class SingleScriptEngineManagerTest {
assertSame(mgr, SingleGremlinScriptEngineManager.instance());
assertSame(mgr, SingleGremlinScriptEngineManager.instance());
assertSame(mgr, SingleGremlinScriptEngineManager.instance());
- assertSame(mgr, SingleGremlinScriptEngineManager.getInstance());
}
@Test(expected = IllegalArgumentException.class)
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinResultSetIntegrateTest.java
----------------------------------------------------------------------
diff --git a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinResultSetIntegrateTest.java b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinResultSetIntegrateTest.java
index 17fd38d..e691e1c 100644
--- a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinResultSetIntegrateTest.java
+++ b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinResultSetIntegrateTest.java
@@ -42,7 +42,7 @@ import org.apache.tinkerpop.gremlin.structure.util.detached.DetachedVertex;
import org.apache.tinkerpop.gremlin.structure.util.detached.DetachedVertexProperty;
import org.apache.tinkerpop.gremlin.structure.util.reference.ReferenceVertex;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph;
-import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistry;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV1d0;
import org.apache.tinkerpop.gremlin.util.iterator.IteratorUtils;
import org.hamcrest.CoreMatchers;
import org.junit.After;
@@ -82,7 +82,7 @@ public class GremlinResultSetIntegrateTest extends AbstractGremlinServerIntegrat
public void beforeTest() {
final MessageSerializer serializer = new GryoMessageSerializerV1d0();
final Map<String,Object> c = new HashMap<>();
- c.put("ioRegistries", Collections.singletonList(TinkerIoRegistry.class.getName()));
+ c.put("ioRegistries", Collections.singletonList(TinkerIoRegistryV1d0.class.getName()));
c.put("custom", Collections.singletonList("groovy.json.JsonBuilder;org.apache.tinkerpop.gremlin.driver.ser.JsonBuilderGryoSerializer"));
serializer.configure(c, null);
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-tools/gremlin-io-test/pom.xml
----------------------------------------------------------------------
diff --git a/gremlin-tools/gremlin-io-test/pom.xml b/gremlin-tools/gremlin-io-test/pom.xml
index 625571f..60a56dc 100644
--- a/gremlin-tools/gremlin-io-test/pom.xml
+++ b/gremlin-tools/gremlin-io-test/pom.xml
@@ -351,14 +351,14 @@ writeSupportedObjects = { mapper, toGryoFunction ->
mapper = GryoMapper.build().
version(GryoVersion.V1_0).
- addRegistry(TinkerIoRegistryV2d0.getInstance()).
+ addRegistry(TinkerIoRegistryV2d0.instance()).
create().createMapper()
writeSupportedObjects(mapper, toGryoV1d0)
mapper = GryoMapper.build().
version(GryoVersion.V3_0).
- addRegistry(TinkerIoRegistryV2d0.getInstance()).
+ addRegistry(TinkerIoRegistryV2d0.instance()).
create().createMapper()
writeSupportedObjects(mapper, toGryoV3d0)
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-tools/gremlin-io-test/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java
----------------------------------------------------------------------
diff --git a/gremlin-tools/gremlin-io-test/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java b/gremlin-tools/gremlin-io-test/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java
index 08eb811..65367dc 100644
--- a/gremlin-tools/gremlin-io-test/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java
+++ b/gremlin-tools/gremlin-io-test/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java
@@ -21,7 +21,7 @@ package org.apache.tinkerpop.gremlin.structure.io.graphson;
import org.apache.tinkerpop.gremlin.driver.ser.AbstractGraphSONMessageSerializerV1d0;
import org.apache.tinkerpop.gremlin.structure.io.AbstractUntypedCompatibilityTest;
import org.apache.tinkerpop.gremlin.structure.io.Compatibility;
-import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistry;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV1d0;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV2d0;
import org.apache.tinkerpop.shaded.jackson.databind.ObjectMapper;
import org.junit.runner.RunWith;
@@ -36,7 +36,7 @@ import java.util.Arrays;
public class GraphSONUntypedCompatibilityTest extends AbstractUntypedCompatibilityTest {
private static ObjectMapper mapperV1 = GraphSONMapper.build().
- addRegistry(TinkerIoRegistry.instance()).
+ addRegistry(TinkerIoRegistryV1d0.instance()).
addCustomModule(new AbstractGraphSONMessageSerializerV1d0.GremlinServerModule()).
version(GraphSONVersion.V1_0).create().createMapper();
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_3/manual-graphson-generator.groovy
----------------------------------------------------------------------
diff --git a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_3/manual-graphson-generator.groovy b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_3/manual-graphson-generator.groovy
index af37164..878f6a3 100644
--- a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_3/manual-graphson-generator.groovy
+++ b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_3/manual-graphson-generator.groovy
@@ -154,7 +154,7 @@ createStaticTraversalMetrics = {
}
mapper = GraphSONMapper.build().
- addRegistry(TinkerIoRegistry.getInstance()).
+ addRegistry(TinkerIoRegistryV1d0.instance()).
addCustomModule(new AbstractGraphSONMessageSerializerV1d0.GremlinServerModule()).
version(GraphSONVersion.V1_0).create().createMapper()
@@ -294,7 +294,7 @@ mapper = GraphSONMapper.build().
}
mapper = GraphSONMapper.build().
- addRegistry(TinkerIoRegistryV2d0.getInstance()).
+ addRegistry(TinkerIoRegistryV2d0.instance()).
typeInfo(TypeInfo.PARTIAL_TYPES).
addCustomModule(GraphSONXModuleV2d0.build().create(false)).
addCustomModule(new org.apache.tinkerpop.gremlin.driver.ser.AbstractGraphSONMessageSerializerV2d0.GremlinServerModule()).
@@ -304,7 +304,7 @@ file = new File("dev-docs/out-graphson-2d0-partial.txt")
file.withWriter { writeSupportedV2Objects(it, mapper, toJsonV2d0PartialTypes) }
mapper = GraphSONMapper.build().
- addRegistry(TinkerIoRegistryV2d0.getInstance()).
+ addRegistry(TinkerIoRegistryV2d0.instance()).
typeInfo(TypeInfo.NO_TYPES).
addCustomModule(GraphSONXModuleV2d0.build().create(false)).
addCustomModule(new org.apache.tinkerpop.gremlin.driver.ser.AbstractGraphSONMessageSerializerV2d0.GremlinServerModule()).
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_4/manual-graphson-generator.groovy
----------------------------------------------------------------------
diff --git a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_4/manual-graphson-generator.groovy b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_4/manual-graphson-generator.groovy
index 1d646f8..e1ff14d 100644
--- a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_4/manual-graphson-generator.groovy
+++ b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphson/_3_2_4/manual-graphson-generator.groovy
@@ -154,7 +154,7 @@ createStaticTraversalMetrics = {
}
mapper = GraphSONMapper.build().
- addRegistry(TinkerIoRegistry.getInstance()).
+ addRegistry(TinkerIoRegistryV1d0.instance()).
addCustomModule(new AbstractGraphSONMessageSerializerV1d0.GremlinServerModule()).
version(GraphSONVersion.V1_0).create().createMapper()
@@ -294,7 +294,7 @@ mapper = GraphSONMapper.build().
}
mapper = GraphSONMapper.build().
- addRegistry(TinkerIoRegistryV2d0.getInstance()).
+ addRegistry(TinkerIoRegistryV2d0.instance()).
typeInfo(TypeInfo.PARTIAL_TYPES).
addCustomModule(GraphSONXModuleV2d0.build().create(false)).
addCustomModule(new org.apache.tinkerpop.gremlin.driver.ser.AbstractGraphSONMessageSerializerV2d0.GremlinServerModule()).
@@ -304,7 +304,7 @@ file = new File("dev-docs/out-graphson-2d0-partial.txt")
file.withWriter { writeSupportedV2Objects(it, mapper, toJsonV2d0PartialTypes) }
mapper = GraphSONMapper.build().
- addRegistry(TinkerIoRegistryV2d0.getInstance()).
+ addRegistry(TinkerIoRegistryV2d0.instance()).
typeInfo(TypeInfo.NO_TYPES).
addCustomModule(GraphSONXModuleV2d0.build().create(false)).
addCustomModule(new org.apache.tinkerpop.gremlin.driver.ser.AbstractGraphSONMessageSerializerV2d0.GremlinServerModule()).
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_3/manual-gryo-generator.groovy
----------------------------------------------------------------------
diff --git a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_3/manual-gryo-generator.groovy b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_3/manual-gryo-generator.groovy
index f641d11..d231aa7 100644
--- a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_3/manual-gryo-generator.groovy
+++ b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_3/manual-gryo-generator.groovy
@@ -192,7 +192,7 @@ writeSupportedObjects = { mapper, toGryoFunction ->
toGryoFunction(ZoneOffset.ofHoursMinutesSeconds(3, 6, 9), "ZoneOffset", mapper)
}
-mapper = GryoMapper.build().addRegistry(TinkerIoRegistryV2d0.getInstance()).create().createMapper()
+mapper = GryoMapper.build().addRegistry(TinkerIoRegistryV2d0.instance()).create().createMapper()
writeSupportedObjects(mapper, toGryoV1d0)
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_4/manual-gryo-generator.groovy
----------------------------------------------------------------------
diff --git a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_4/manual-gryo-generator.groovy b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_4/manual-gryo-generator.groovy
index d204d4e..6e66f36 100644
--- a/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_4/manual-gryo-generator.groovy
+++ b/gremlin-tools/gremlin-io-test/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/gryo/_3_2_4/manual-gryo-generator.groovy
@@ -192,7 +192,7 @@ writeSupportedObjects = { mapper, toGryoFunction ->
toGryoFunction(ZoneOffset.ofHoursMinutesSeconds(3, 6, 9), "ZoneOffset", mapper)
}
-mapper = GryoMapper.build().addRegistry(TinkerIoRegistryV2d0.getInstance()).create().createMapper()
+mapper = GryoMapper.build().addRegistry(TinkerIoRegistryV2d0.instance()).create().createMapper()
writeSupportedObjects(mapper, toGryoV1d0)
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/structure/HadoopGraph.java
----------------------------------------------------------------------
diff --git a/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/structure/HadoopGraph.java b/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/structure/HadoopGraph.java
index fdcfef2..63c7c57 100644
--- a/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/structure/HadoopGraph.java
+++ b/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/structure/HadoopGraph.java
@@ -491,11 +491,6 @@ public final class HadoopGraph implements Graph {
public Features.VertexPropertyFeatures properties() {
return new Features.VertexPropertyFeatures() {
@Override
- public boolean supportsAddProperty() {
- return false;
- }
-
- @Override
public boolean supportsRemoveProperty() {
return false;
}
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/jsr223/TinkerGraphGremlinPlugin.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/jsr223/TinkerGraphGremlinPlugin.java b/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/jsr223/TinkerGraphGremlinPlugin.java
index 55e9a4d..b7b15d9 100644
--- a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/jsr223/TinkerGraphGremlinPlugin.java
+++ b/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/jsr223/TinkerGraphGremlinPlugin.java
@@ -34,7 +34,7 @@ import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerFactory;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraphVariables;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerHelper;
-import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistry;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV1d0;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV2d0;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerProperty;
import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerVertex;
@@ -53,7 +53,7 @@ public final class TinkerGraphGremlinPlugin extends AbstractGremlinPlugin {
TinkerGraph.class,
TinkerGraphVariables.class,
TinkerHelper.class,
- TinkerIoRegistry.class,
+ TinkerIoRegistryV1d0.class,
TinkerIoRegistryV2d0.class,
TinkerProperty.class,
TinkerVertex.class,
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistry.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistry.java b/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistry.java
deleted file mode 100644
index 5abb363..0000000
--- a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistry.java
+++ /dev/null
@@ -1,272 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-package org.apache.tinkerpop.gremlin.tinkergraph.structure;
-
-import org.apache.commons.configuration.BaseConfiguration;
-import org.apache.commons.configuration.Configuration;
-import org.apache.tinkerpop.gremlin.structure.Edge;
-import org.apache.tinkerpop.gremlin.structure.Vertex;
-import org.apache.tinkerpop.gremlin.structure.io.AbstractIoRegistry;
-import org.apache.tinkerpop.gremlin.structure.io.GraphReader;
-import org.apache.tinkerpop.gremlin.structure.io.GraphWriter;
-import org.apache.tinkerpop.gremlin.structure.io.IoRegistry;
-import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONIo;
-import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONTokens;
-import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONUtil;
-import org.apache.tinkerpop.gremlin.structure.io.gryo.GryoIo;
-import org.apache.tinkerpop.gremlin.structure.io.gryo.GryoReader;
-import org.apache.tinkerpop.gremlin.structure.io.gryo.GryoWriter;
-import org.apache.tinkerpop.gremlin.structure.util.Attachable;
-import org.apache.tinkerpop.gremlin.structure.util.detached.DetachedEdge;
-import org.apache.tinkerpop.gremlin.structure.util.detached.DetachedVertex;
-import org.apache.tinkerpop.shaded.jackson.core.JsonGenerator;
-import org.apache.tinkerpop.shaded.jackson.core.JsonParser;
-import org.apache.tinkerpop.shaded.jackson.core.JsonProcessingException;
-import org.apache.tinkerpop.shaded.jackson.databind.DeserializationContext;
-import org.apache.tinkerpop.shaded.jackson.databind.SerializerProvider;
-import org.apache.tinkerpop.shaded.jackson.databind.deser.std.StdDeserializer;
-import org.apache.tinkerpop.shaded.jackson.databind.jsontype.TypeSerializer;
-import org.apache.tinkerpop.shaded.jackson.databind.module.SimpleModule;
-import org.apache.tinkerpop.shaded.jackson.databind.ser.std.StdSerializer;
-import org.apache.tinkerpop.shaded.kryo.Kryo;
-import org.apache.tinkerpop.shaded.kryo.Serializer;
-import org.apache.tinkerpop.shaded.kryo.io.Input;
-import org.apache.tinkerpop.shaded.kryo.io.Output;
-
-import java.io.ByteArrayInputStream;
-import java.io.ByteArrayOutputStream;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-
-/**
- * An implementation of the {@link IoRegistry} interface that provides serializers with custom configurations for
- * implementation specific classes that might need to be serialized. This registry allows a {@link TinkerGraph} to
- * be serialized directly which is useful for moving small graphs around on the network.
- * <p/>
- * Most providers need not implement this kind of custom serializer as they will deal with much larger graphs that
- * wouldn't be practical to serialize in this fashion. This is a bit of a special case for TinkerGraph given its
- * in-memory status. Typical implementations would create serializers for a complex vertex identifier or a
- * custom data class like a "geographic point".
- *
- * @author Stephen Mallette (http://stephen.genoprime.com)
- * @deprecated As of release 3.2.4, replaced by {@link TinkerIoRegistryV1d0}.
- */
-@Deprecated
-public final class TinkerIoRegistry extends AbstractIoRegistry {
-
- private static final TinkerIoRegistry INSTANCE = new TinkerIoRegistry();
-
- private TinkerIoRegistry() {
- register(GryoIo.class, TinkerGraph.class, new TinkerGraphGryoSerializer());
- register(GraphSONIo.class, null, new TinkerModule());
- }
-
- /**
- * @deprecated As of release 3.2.4, replaced by {@link #instance()}.
- */
- @Deprecated
- public static TinkerIoRegistry getInstance() {
- return instance();
- }
-
- public static TinkerIoRegistry instance() {
- return INSTANCE;
- }
-
- /**
- * Provides a method to serialize an entire {@link TinkerGraph} into itself for Gryo. This is useful when
- * shipping small graphs around through Gremlin Server. Reuses the existing Kryo instance for serialization.
- */
- final static class TinkerGraphGryoSerializer extends Serializer<TinkerGraph> {
- @Override
- public void write(final Kryo kryo, final Output output, final TinkerGraph graph) {
- try (final ByteArrayOutputStream stream = new ByteArrayOutputStream()) {
- GryoWriter.build().mapper(() -> kryo).create().writeGraph(stream, graph);
- final byte[] bytes = stream.toByteArray();
- output.writeInt(bytes.length);
- output.write(bytes);
- } catch (Exception io) {
- throw new RuntimeException(io);
- }
- }
-
- @Override
- public TinkerGraph read(final Kryo kryo, final Input input, final Class<TinkerGraph> tinkerGraphClass) {
- final Configuration conf = new BaseConfiguration();
- conf.setProperty("gremlin.tinkergraph.defaultVertexPropertyCardinality", "list");
- final TinkerGraph graph = TinkerGraph.open(conf);
- final int len = input.readInt();
- final byte[] bytes = input.readBytes(len);
- try (final ByteArrayInputStream stream = new ByteArrayInputStream(bytes)) {
- GryoReader.build().mapper(() -> kryo).create().readGraph(stream, graph);
- } catch (Exception io) {
- throw new RuntimeException(io);
- }
-
- return graph;
- }
- }
-
- /**
- * Provides a method to serialize an entire {@link TinkerGraph} into itself for GraphSON. This is useful when
- * shipping small graphs around through Gremlin Server.
- */
- final static class TinkerModule extends SimpleModule {
- public TinkerModule() {
- super("tinkergraph-1.0");
- addSerializer(TinkerGraph.class, new TinkerGraphJacksonSerializer());
- addDeserializer(TinkerGraph.class, new TinkerGraphJacksonDeserializer());
- }
- }
-
- /**
- * Serializes the graph into an edge list format. Edge list is a better choices than adjacency list (which is
- * typically standard from the {@link GraphReader} and {@link GraphWriter} perspective) in this case because
- * the use case for this isn't around massive graphs. The use case is for "small" subgraphs that are being
- * shipped over the wire from Gremlin Server. Edge list format is a bit easier for non-JVM languages to work
- * with as a format and doesn't require a cache for loading (as vertex labels are not serialized in adjacency
- * list).
- */
- final static class TinkerGraphJacksonSerializer extends StdSerializer<TinkerGraph> {
-
- public TinkerGraphJacksonSerializer() {
- super(TinkerGraph.class);
- }
-
- @Override
- public void serialize(final TinkerGraph graph, final JsonGenerator jsonGenerator, final SerializerProvider serializerProvider)
- throws IOException {
- jsonGenerator.writeStartObject();
-
- jsonGenerator.writeFieldName(GraphSONTokens.VERTICES);
- jsonGenerator.writeStartArray();
-
- final Iterator<Vertex> vertices = graph.vertices();
- while (vertices.hasNext()) {
- serializerProvider.defaultSerializeValue(vertices.next(), jsonGenerator);
- }
-
- jsonGenerator.writeEndArray();
-
- jsonGenerator.writeFieldName(GraphSONTokens.EDGES);
- jsonGenerator.writeStartArray();
-
- final Iterator<Edge> edges = graph.edges();
- while (edges.hasNext()) {
- serializerProvider.defaultSerializeValue(edges.next(), jsonGenerator);
- }
-
- jsonGenerator.writeEndArray();
-
- jsonGenerator.writeEndObject();
- }
-
- @Override
- public void serializeWithType(final TinkerGraph graph, final JsonGenerator jsonGenerator,
- final SerializerProvider serializerProvider, final TypeSerializer typeSerializer) throws IOException {
- jsonGenerator.writeStartObject();
- jsonGenerator.writeStringField(GraphSONTokens.CLASS, TinkerGraph.class.getName());
-
- jsonGenerator.writeFieldName(GraphSONTokens.VERTICES);
- jsonGenerator.writeStartArray();
- jsonGenerator.writeString(ArrayList.class.getName());
- jsonGenerator.writeStartArray();
-
- final Iterator<Vertex> vertices = graph.vertices();
- while (vertices.hasNext()) {
- GraphSONUtil.writeWithType(vertices.next(), jsonGenerator, serializerProvider, typeSerializer);
- }
-
- jsonGenerator.writeEndArray();
- jsonGenerator.writeEndArray();
-
- jsonGenerator.writeFieldName(GraphSONTokens.EDGES);
- jsonGenerator.writeStartArray();
- jsonGenerator.writeString(ArrayList.class.getName());
- jsonGenerator.writeStartArray();
-
- final Iterator<Edge> edges = graph.edges();
- while (edges.hasNext()) {
- GraphSONUtil.writeWithType(edges.next(), jsonGenerator, serializerProvider, typeSerializer);
- }
-
- jsonGenerator.writeEndArray();
- jsonGenerator.writeEndArray();
-
- jsonGenerator.writeEndObject();
- }
- }
-
- /**
- * Deserializes the edge list format.
- */
- static class TinkerGraphJacksonDeserializer extends StdDeserializer<TinkerGraph> {
- public TinkerGraphJacksonDeserializer() {
- super(TinkerGraph.class);
- }
-
- @Override
- public TinkerGraph deserialize(JsonParser jsonParser, DeserializationContext deserializationContext) throws IOException, JsonProcessingException {
- final Configuration conf = new BaseConfiguration();
- conf.setProperty("gremlin.tinkergraph.defaultVertexPropertyCardinality", "list");
- final TinkerGraph graph = TinkerGraph.open(conf);
-
- final List<Map<String, Object>> edges;
- final List<Map<String, Object>> vertices;
- if (!jsonParser.getCurrentToken().isStructStart()) {
- if (!jsonParser.getCurrentName().equals(GraphSONTokens.VERTICES))
- throw new IOException(String.format("Expected a '%s' key", GraphSONTokens.VERTICES));
-
- jsonParser.nextToken();
- vertices = (List<Map<String, Object>>) deserializationContext.readValue(jsonParser, ArrayList.class);
- jsonParser.nextToken();
-
- if (!jsonParser.getCurrentName().equals(GraphSONTokens.EDGES))
- throw new IOException(String.format("Expected a '%s' key", GraphSONTokens.EDGES));
-
- jsonParser.nextToken();
- edges = (List<Map<String, Object>>) deserializationContext.readValue(jsonParser, ArrayList.class);
- } else {
- final Map<String, Object> graphData = deserializationContext.readValue(jsonParser, HashMap.class);
- vertices = (List<Map<String,Object>>) graphData.get(GraphSONTokens.VERTICES);
- edges = (List<Map<String,Object>>) graphData.get(GraphSONTokens.EDGES);
- }
-
- for (Map<String, Object> vertexData : vertices) {
- final DetachedVertex detached = new DetachedVertex(vertexData.get(GraphSONTokens.ID),
- vertexData.get(GraphSONTokens.LABEL).toString(), (Map<String,Object>) vertexData.get(GraphSONTokens.PROPERTIES));
- detached.attach(Attachable.Method.getOrCreate(graph));
- }
-
- for (Map<String, Object> edgeData : edges) {
- final DetachedEdge detached = new DetachedEdge(edgeData.get(GraphSONTokens.ID),
- edgeData.get(GraphSONTokens.LABEL).toString(), (Map<String,Object>) edgeData.get(GraphSONTokens.PROPERTIES),
- edgeData.get(GraphSONTokens.OUT), edgeData.get(GraphSONTokens.OUT_LABEL).toString(),
- edgeData.get(GraphSONTokens.IN), edgeData.get(GraphSONTokens.IN_LABEL).toString());
- detached.attach(Attachable.Method.getOrCreate(graph));
- }
-
- return graph;
- }
- }
-}
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistryV2d0.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistryV2d0.java b/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistryV2d0.java
index 280d1af..74c128e 100644
--- a/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistryV2d0.java
+++ b/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerIoRegistryV2d0.java
@@ -74,14 +74,6 @@ public final class TinkerIoRegistryV2d0 extends AbstractIoRegistry {
register(GraphSONIo.class, null, new TinkerModuleV2d0());
}
- /**
- * @deprecated As of release 3.2.4, replaced by {@link #instance()}.
- */
- @Deprecated
- public static TinkerIoRegistryV2d0 getInstance() {
- return INSTANCE;
- }
-
public static TinkerIoRegistryV2d0 instance() {
return INSTANCE;
}
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerTest.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerTest.java b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerTest.java
deleted file mode 100644
index 703d127..0000000
--- a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerTest.java
+++ /dev/null
@@ -1,80 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-package org.apache.tinkerpop.gremlin.tinkergraph.structure;
-
-import org.apache.tinkerpop.gremlin.structure.io.gryo.GryoMapper;
-import org.apache.tinkerpop.shaded.kryo.Kryo;
-import org.apache.tinkerpop.shaded.kryo.Registration;
-import org.apache.tinkerpop.shaded.kryo.io.Input;
-import org.apache.tinkerpop.shaded.kryo.io.Output;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.Mock;
-import org.mockito.runners.MockitoJUnitRunner;
-
-import java.util.Arrays;
-
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.anyInt;
-import static org.mockito.Mockito.atLeastOnce;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
-/**
- * Unit tests for {@link TinkerIoRegistry.TinkerGraphGryoSerializer}
- */
-@RunWith(MockitoJUnitRunner.class)
-public class TinkerGraphGryoSerializerTest {
-
- @Mock
- private Kryo kryo;
- @Mock
- private Registration registration;
- @Mock
- private Output output;
- @Mock
- private Input input;
-
- private TinkerGraph graph = TinkerGraph.open();
- private TinkerIoRegistry.TinkerGraphGryoSerializer serializer = new TinkerIoRegistry.TinkerGraphGryoSerializer();
-
- @Before
- public void setUp() throws Exception {
- when(kryo.getRegistration((Class) any())).thenReturn(registration);
- when(input.readBytes(anyInt())).thenReturn(Arrays.copyOf(GryoMapper.HEADER, 100));
- }
-
- @Test
- public void shouldVerifyKryoUsedForWrite() throws Exception {
- serializer.write(kryo, output, graph);
- verify(kryo, atLeastOnce()).getRegistration((Class) any());
- }
-
- @Test
- public void shouldVerifyKryoUsedForRead() throws Exception {
- // Not possible to mock an entire deserialization so just verify the same kryo instances are being used
- try {
- serializer.read(kryo, input, TinkerGraph.class);
- } catch (RuntimeException ex) {
- verify(kryo, atLeastOnce()).readObject(any(), any());
- verify(kryo, atLeastOnce()).readClassAndObject(any());
- }
- }
-}
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/1a1ba583/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerV1d0Test.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerV1d0Test.java b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerV1d0Test.java
index 2233d53..fff3be6 100644
--- a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerV1d0Test.java
+++ b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphGryoSerializerV1d0Test.java
@@ -19,9 +19,6 @@
package org.apache.tinkerpop.gremlin.tinkergraph.structure;
import org.apache.tinkerpop.gremlin.structure.io.gryo.GryoMapper;
-import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph;
-import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistry;
-import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV1d0;
import org.apache.tinkerpop.shaded.kryo.Kryo;
import org.apache.tinkerpop.shaded.kryo.Registration;
import org.apache.tinkerpop.shaded.kryo.io.Input;
@@ -41,7 +38,7 @@ import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
/**
- * Unit tests for {@link TinkerIoRegistry.TinkerGraphGryoSerializer}
+ * Unit tests for {@link TinkerIoRegistryV1d0.TinkerGraphGryoSerializer}
*/
@RunWith(MockitoJUnitRunner.class)
public class TinkerGraphGryoSerializerV1d0Test {
[2/5] tinkerpop git commit: TINKERPOP-1094 Removed deprecated
VertexPropertyFeatures - addProperty
Posted by sp...@apache.org.
TINKERPOP-1094 Removed deprecated VertexPropertyFeatures - addProperty
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/732c80b0
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/732c80b0
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/732c80b0
Branch: refs/heads/TINKERPOP-929
Commit: 732c80b001a098cae15999cbc382180f4024cd65
Parents: 4d0a5fb
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Fri Jun 9 10:10:12 2017 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Fri Jun 9 10:10:12 2017 -0400
----------------------------------------------------------------------
CHANGELOG.asciidoc | 1 +
docs/src/upgrade/release-3.3.x.asciidoc | 4 +++-
.../apache/tinkerpop/gremlin/structure/Graph.java | 18 ------------------
3 files changed, 4 insertions(+), 19 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/732c80b0/CHANGELOG.asciidoc
----------------------------------------------------------------------
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index aa3ca46..404a570 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -32,6 +32,7 @@ TinkerPop 3.3.0 (Release Date: NOT OFFICIALLY RELEASED YET)
* Added "attachment requisite" `VertexProperty.element()` and `Property.element()` data in GraphSON serialization.
* GraphSON 3.0 is now the default serialization format in TinkerGraph and Gremlin Server.
* Established the GraphSON 3.0 format.
+* Removed previously deprecated `VertexPropertyFeatures.supportsAddProperty()`.
* Removed previously deprecated TinkerGraph configuration member variables.
* Removed previously deprecated `OpSelectorHandler.errorMeter` and `AbstractEvalOpProcessor.errorMeter` fields.
* Removed previously deprecated `AbstractEvalOpProcessor.validBindingName` field.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/732c80b0/docs/src/upgrade/release-3.3.x.asciidoc
----------------------------------------------------------------------
diff --git a/docs/src/upgrade/release-3.3.x.asciidoc b/docs/src/upgrade/release-3.3.x.asciidoc
index 800fd7a..8d1c712 100644
--- a/docs/src/upgrade/release-3.3.x.asciidoc
+++ b/docs/src/upgrade/release-3.3.x.asciidoc
@@ -192,6 +192,8 @@ The following deprecated classes, methods or fields have been removed in this ve
** `org.apache.tinkerpop.gremlin.jsr223.GremlinModule`
** `org.apache.tinkerpop.gremlin.jsr223.GremlinScriptEngineManager#addModule(GremlinModule)`
** `org.apache.tinkerpop.gremlin.jsr223.console.PluginAcceptor`
+** `org.apache.tinkerpop.gremlin.structure.Graph.Features.VertexPropertyFeatures#supportsAddProperty()`
+** `org.apache.tinkerpop.gremlin.structure.Graph.Features.VertexPropertyFeatures#FEATURE_ADD_PROPERTY`
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_PROCESS_STANDARD`
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_PROCESS_COMPUTER`
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_ENVIRONMENT`
@@ -244,7 +246,7 @@ The following deprecated classes, methods or fields have been removed in this ve
** `org.apache.tinkerpop.gremlin.spark.groovy.plugin.SparkGremlinPlugin`
* `tinkergraph-gremlin`
** `org.apache.tinkerpop.gremlin.tinkergraph.groovy.plugin.TinkerGraphGremlinPlugin`
-** `org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph.CONFIG_*`
+** `org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph#CONFIG_*`
Please see the javadoc deprecation notes or upgrade documentation specific to when the deprecation took place to
understand how to resolve this breaking change.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/732c80b0/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java
index 4e5395a..3c9ce8e 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java
@@ -751,12 +751,6 @@ public interface Graph extends AutoCloseable, Host {
* Features that are related to {@link Vertex} {@link Property} objects.
*/
public interface VertexPropertyFeatures extends PropertyFeatures {
- /**
- * @deprecated As of release 3.1.1-incubating, replaced by
- * {@link org.apache.tinkerpop.gremlin.structure.Graph.Features.VertexFeatures#FEATURE_META_PROPERTIES}
- */
- @Deprecated
- public static final String FEATURE_ADD_PROPERTY = "AddProperty";
public static final String FEATURE_REMOVE_PROPERTY = "RemoveProperty";
public static final String FEATURE_USER_SUPPLIED_IDS = "UserSuppliedIds";
public static final String FEATURE_NUMERIC_IDS = "NumericIds";
@@ -766,18 +760,6 @@ public interface Graph extends AutoCloseable, Host {
public static final String FEATURE_ANY_IDS = "AnyIds";
/**
- * Determines if a {@link VertexProperty} allows properties to be added.
- *
- * @deprecated As of release 3.1.1-incubating, replaced by
- * {@link org.apache.tinkerpop.gremlin.structure.Graph.Features.VertexFeatures#supportsMetaProperties()}
- */
- @Deprecated
- @FeatureDescriptor(name = FEATURE_ADD_PROPERTY)
- public default boolean supportsAddProperty() {
- return true;
- }
-
- /**
* Determines if a {@link VertexProperty} allows properties to be removed.
*/
@FeatureDescriptor(name = FEATURE_REMOVE_PROPERTY)
[4/5] tinkerpop git commit: TINKERPOP-1316 Removed deprecated
GryoMessageSerializer constructor
Posted by sp...@apache.org.
TINKERPOP-1316 Removed deprecated GryoMessageSerializer constructor
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/89653bbc
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/89653bbc
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/89653bbc
Branch: refs/heads/TINKERPOP-929
Commit: 89653bbcbfc3633092cb07a7bddf69aff8fdf572
Parents: 1a1ba58
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Fri Jun 9 11:03:33 2017 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Fri Jun 9 11:03:33 2017 -0400
----------------------------------------------------------------------
CHANGELOG.asciidoc | 1 +
docs/src/upgrade/release-3.3.x.asciidoc | 2 ++
.../gremlin/driver/ser/GryoMessageSerializerV1d0.java | 11 -----------
3 files changed, 3 insertions(+), 11 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/89653bbc/CHANGELOG.asciidoc
----------------------------------------------------------------------
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index 1de020f..ad08fe1 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -32,6 +32,7 @@ TinkerPop 3.3.0 (Release Date: NOT OFFICIALLY RELEASED YET)
* Added "attachment requisite" `VertexProperty.element()` and `Property.element()` data in GraphSON serialization.
* GraphSON 3.0 is now the default serialization format in TinkerGraph and Gremlin Server.
* Established the GraphSON 3.0 format.
+* Removed previously deprecated `GryoMessageSerializerV1d0(GryoMapper)` constructor.
* Removed previously deprecated `TinkerIoRegistry`.
* Removed previously deprecated `getInstance()` methods on all TinkerPop classes.
* Removed previously deprecated `VertexPropertyFeatures.supportsAddProperty()`.
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/89653bbc/docs/src/upgrade/release-3.3.x.asciidoc
----------------------------------------------------------------------
diff --git a/docs/src/upgrade/release-3.3.x.asciidoc b/docs/src/upgrade/release-3.3.x.asciidoc
index 7d90e89..85a4f53 100644
--- a/docs/src/upgrade/release-3.3.x.asciidoc
+++ b/docs/src/upgrade/release-3.3.x.asciidoc
@@ -199,6 +199,8 @@ The following deprecated classes, methods or fields have been removed in this ve
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_PROCESS_COMPUTER`
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_ENVIRONMENT`
** `org.apache.tinkerpop.gremlin.structure.Graph.OptIn#SUITE_GROOVY_ENVIRONMENT_INTEGRATE`
+* `gremlin-driver`
+** `org.apache.tinkerpop.gremlin.driver.ser.GryoMessageSerializerV1d0(GryoMapper)`
* `gremlin-groovy`
** `org.apache.tinkerpop.gremlin.groovy.AbstractImportCustomizerProvider`
** `org.apache.tinkerpop.gremlin.groovy.CompilerCustomizerProvider`
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/89653bbc/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GryoMessageSerializerV1d0.java
----------------------------------------------------------------------
diff --git a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GryoMessageSerializerV1d0.java b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GryoMessageSerializerV1d0.java
index 445b6cc..9b9f1f3 100644
--- a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GryoMessageSerializerV1d0.java
+++ b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GryoMessageSerializerV1d0.java
@@ -50,17 +50,6 @@ public final class GryoMessageSerializerV1d0 extends AbstractGryoMessageSerializ
super(kryo.create());
}
- /**
- * Creates an instance with a standard {@link GryoMapper} instance. Note that the instance supplied
- * will be overridden by {@link #configure} if it is called.
- *
- * @deprecated As of release 3.2.1, replaced by {@link GryoMessageSerializerV1d0#GryoMessageSerializerV1d0(GryoMapper.Builder)}.
- */
- @Deprecated
- public GryoMessageSerializerV1d0(final GryoMapper kryo) {
- super(kryo);
- }
-
@Override
public String[] mimeTypesSupported() {
return new String[]{serializeToString ? MIME_TYPE_STRINGD : MIME_TYPE};