updated upgrade docs and CHANGELOG. Removed all the 'breaking'-tags in the 3.3.0 CHANGELOG listing for @spmallette.
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/16180e18 Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/16180e18 Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/16180e18 Branch: refs/heads/master Commit: 16180e1811038f388fcfbdaf312f36fe2c59f965 Parents: a492f87 Author: Marko A. Rodriguez <[email protected]> Authored: Thu Nov 24 05:04:54 2016 -0700 Committer: Marko A. Rodriguez <[email protected]> Committed: Thu Nov 24 05:04:54 2016 -0700 ---------------------------------------------------------------------- CHANGELOG.asciidoc | 6 +++--- docs/src/upgrade/release-3.3.x.asciidoc | 16 ++++++++++++++-- 2 files changed, 17 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/16180e18/CHANGELOG.asciidoc ---------------------------------------------------------------------- diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc index dba4645..498ef23 100644 --- a/CHANGELOG.asciidoc +++ b/CHANGELOG.asciidoc @@ -30,11 +30,11 @@ TinkerPop 3.3.0 (Release Date: NOT OFFICIALLY RELEASED YET) * Added `Vertex`, `Edge`, `VertexProperty`, and `Property` serializers to Gremlin-Python and exposed tests that use graph object arguments. * `Bytecode.getSourceInstructions()` and `Bytecode.getStepInstructions()` now returns `List<Instruction>` instead of `Iterable<Instruction>`. * Added various `TraversalStrategy` registrations with `GryoMapper`. -* Fixed a naming mistake in Gremlin-Python: `IdentityRemoveStrategy` is now called `IdentityRemovalStrategy`. (*breaking*) +* Fixed a naming mistake in Gremlin-Python: `IdentityRemoveStrategy` is now called `IdentityRemovalStrategy`. * Added `TranslationStrategy` test infrastructure that verifies `Bytecode` generated from a translation is equal to the original `Bytecode`. * Moved `NumberHelper` into the `org.apache.tinkerpop.gremlin.util` package. * Added `Pop.mixed` instead of using `null` to represent such semantics. -* `select()`-step now defaults to using `Pop.last` instead of `Pop.mixed`. (*breaking*) +* `select()`-step now defaults to using `Pop.last` instead of `Pop.mixed`. * Removed previously deprecated `Console` constructor that took a `String` as an argument from `gremlin-console`. * Removed previously deprecated `ConcurrentBindings` from `gremlin-groovy`. * Removed previously deprecated `ScriptExecutor` from `gremlin-groovy`. @@ -42,7 +42,7 @@ TinkerPop 3.3.0 (Release Date: NOT OFFICIALLY RELEASED YET) * Removed previously deprecated `GremlinGroovyScriptEngine` constructor that took `ImportCustomizerProvider` as an argument from `gremlin-groovy`. * Removed previously deprecated `GremlinGroovyScriptEngine#plugins()` from `gremlin-groovy`. * Added `OptionalStep` for use with `optional()` to better handle issues associated with branch side-effects. -* `UnfoldStep` now supports unfolding of arrays. (*breaking*) +* `UnfoldStep` now supports unfolding of arrays. * Removed all performance tests that were not part of `gremlin-benchmark`. * Removed dependency on `junit-benchmarks` and it's related reference to `h2`. * Moved the source for the "home page" into the repository under `/site` so that it easier to accept contributions. http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/16180e18/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 c0213d8..e1f42da 100644 --- a/docs/src/upgrade/release-3.3.x.asciidoc +++ b/docs/src/upgrade/release-3.3.x.asciidoc @@ -32,6 +32,17 @@ Please see the link:https://github.com/apache/tinkerpop/blob/3.3.3/CHANGELOG.asc Upgrading for Users ~~~~~~~~~~~~~~~~~~~ +GraphTraversal Has-Methods Re-Organized +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ + +`GraphTraversal.hasXXX()`, where `XXX` is `Id`, `Label`, `Key`, `Value`, was faulty in that they relied on calling an +intermediate method for flattening `Object[]` arguments and thus, yielding a non 1-to-1 correspondence between `GraphTraversal` +and `Bytecode`. This has been remedied. Most users will not notice this change. Perhaps only some users that may use +Java reflection over `GraphTraversal` might have a simple problem. + + +See: link:https://issues.apache.org/jira/browse/TINKERPOP-1520[TINKERPOP-1520] + SelectStep Defaults to Pop.last ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ @@ -47,16 +58,17 @@ Assuming that `x` is not a `Pop` argument: If an explicit `Pop` argument is provided, then no changes are required. -See: https://issues.apache.org/jira/browse/TINKERPOP-1541[TINKERPOP-1541] +See: link:https://issues.apache.org/jira/browse/TINKERPOP-1541[TINKERPOP-1541] OptionalStep and Side-Effects ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ The `optional()`-step was previously implemented using `ChooseStep`. However, if the optional branch contained side-effects, -then unexpected behaviors can emerge. Thus, a potential backwards compatibility issue arises is side-effects were being +then unexpected behaviors can emerge. Thus, a potential backwards compatibility issue arises if side-effects were being used in `optional()`. However, the behavior would be unpredictable so this backwards incompatibility is desirable. See link:https://issues.apache.org/jira/browse/TINKERPOP-1506[TINKERPOP-1506] + Gremlin Console Script Execution ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
