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 2019/11/08 12:23:58 UTC

[tinkerpop] branch master updated (4240b35 -> a2dfcd0)

This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git.


    from 4240b35  Merge branch 'tp34'
     add 1088cd9  [TINKERPOP-2274] Fixed the issues revealed while compiling and running tests under Windows
     add 9cd001b  [TINKERPOP-2274] Changed back the default persistent use of Spark Context for tests, still making it non-periststent for the tests that are reveal locks under Windows. This should speed up the integration tests back too.
     add ec62bae  Update pom.xml
     add 26a2dfd  Merge branch 'TINKERPOP-2274-clean' of https://github.com/rpopov/tinkerpop into TINKERPOP-2274-clean
     add e061038  [TINKERPOP-2274] Made the FileSystemCheck tests use non-persistent Spark context this way avoiding locks in the file system.
     add eeb7e11  [TINKERPOP-2274] Fixed the use of a directory in place of file that had failed the integration tests. Fixed.
     add d339ba3  [TINKERPOP-2274] Split the TestHelper from gremlin-core/test into two classes - CoreTestHelper in the gremlin-core/main, so that it poublishes common methods, available to be inherited, whereas the only duplicated code is left in core/test TestHelper not published, thus avoiding name conflict with TestHelper from gremlin-test
     add 954f1a5  [TINKERPOP-2274] Removed references to CoreTestHelper outsde the gremlin-core project
     add 5c4cace  Squashed commit of the following:
     add ff071b6  Merge branch 'pr-1188' into tp33
     add 5a3535d  Minor syntax error caused :install command to fail CTR
     add 88578a8  Updated javadoc for the CoreTestHelper
     add 5e43fdc  Aligned maven version to what was previously enforced - 3.3.9
     add a40bc3d  TestHelper.RANDOM should be used in all tests so the seed gets captured
     add ef941e3  Merge branch 'tp33' into tp34
     new a2dfcd0  Merge branch 'tp34'

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 docs/preprocessor/install-plugins.groovy           |   2 +-
 gremlin-archetype/pom.xml                          |   1 +
 .../tinkerpop/gremlin/console/ConsoleFs.groovy     |   8 +-
 .../console/commands/UninstallCommand.groovy       |  12 +-
 .../jsr223/DriverRemoteAcceptorIntegrateTest.java  |  25 +--
 .../console/jsr223/DriverRemoteAcceptorTest.java   |   4 +-
 gremlin-core/pom.xml                               |   6 +
 .../tinkerpop/gremlin/structure/io/Storage.java    | 101 +++++++++--
 .../tinkerpop/gremlin/util/CoreTestHelper.java     | 188 +++++++++++++++++++++
 .../org/apache/tinkerpop/gremlin/AssertHelper.java |  20 ++-
 .../org/apache/tinkerpop/gremlin/TestHelper.java   | 147 ----------------
 .../jsr223/DefaultScriptCustomizerTest.java        |   6 +-
 .../gremlin/jsr223/ScriptEngineCacheTest.java      |   8 +-
 .../jsr223/ScriptFileGremlinPluginTest.java        |  11 +-
 .../gremlin/process/traversal/OrderTest.java       |   3 +-
 .../optimization/EarlyLimitStrategyTest.java       |   4 +-
 .../optimization/OrderLimitStrategyTest.java       |   2 +-
 .../StandardVerificationStrategyTest.java          |   6 +-
 .../gremlin/structure/io/GraphMigratorTest.java    |   4 +-
 .../tinkerpop/gremlin/structure/io/IoCoreTest.java |   4 +-
 .../structure/io/graphson/GraphSONTokensTest.java  |   4 +-
 .../structure/io/graphson/GraphSONUtilTest.java    |   4 +-
 .../gremlin/structure/util/ComparatorsTest.java    |   4 +-
 .../gremlin/structure/util/ElementHelperTest.java  |   4 +-
 .../gremlin/structure/util/GraphFactoryTest.java   |  32 ++--
 .../structure/util/GraphVariableHelperTest.java    |   4 +-
 .../gremlin/structure/util/StringFactoryTest.java  |   4 +-
 .../apache/tinkerpop/gremlin/util/GremlinTest.java |   4 +-
 .../tinkerpop/gremlin/util/SerializerTest.java     |   4 +-
 .../tinkerpop/gremlin/util/TimeUtilTest.java       |   4 +-
 .../gremlin/util/function/FunctionUtilsTest.java   |   5 +-
 .../gremlin/util/iterator/IteratorUtilsTest.java   |   4 +-
 .../tinkerpop/gremlin/util/tools/MultiMapTest.java |   5 +-
 .../gremlin/groovy/util/DependencyGrabber.groovy   |  74 ++++----
 .../gremlin/groovy/engine/GremlinExecutorTest.java |   4 +-
 .../GremlinGroovyScriptEngineFileSandboxTest.java  |   5 +-
 .../util/DependencyGrabberIntegrateTest.java       |  76 +++++----
 .../gremlin/util/TinkerGraphProvider.java          |   4 +-
 .../gremlin/server/util/GremlinServerInstall.java  |   7 +-
 .../gremlin/server/GremlinDriverIntegrateTest.java |   6 +-
 .../tinkerpop/gremlin/server/ServerTestHelper.java |  57 +++++--
 .../gremlin/util/Log4jRecordingAppender.java       |  17 +-
 .../gremlin/AbstractFileGraphProvider.java         |  12 +-
 .../apache/tinkerpop/gremlin/GraphProvider.java    |   3 +-
 .../org/apache/tinkerpop/gremlin/TestHelper.java   | 142 ++--------------
 .../gremlin/jsr223/BindingsScriptEngineTest.java   |   3 +-
 .../gremlin/structure/io/IoGraphTest.java          |   8 +-
 .../tinkerpop/gremlin/structure/io/IoTest.java     |  31 ++--
 hadoop-gremlin/README.md                           |  52 ++++++
 hadoop-gremlin/pom.xml                             |  29 ++++
 .../apache/tinkerpop/gremlin/hadoop/Constants.java |  22 +++
 .../computer/AbstractHadoopGraphComputer.java      |  12 +-
 .../hadoop/structure/io/FileSystemStorage.java     | 101 ++++++-----
 .../hadoop/structure/io/InputOutputHelper.java     |   2 +-
 .../computer/AbstractHadoopGraphComputerTest.java  |  59 ++++---
 .../structure/io/RecordReaderWriterTest.java       |   5 +-
 pom.xml                                            |  18 +-
 .../tinkerpop/gremlin/spark/structure/Spark.java   |   7 +-
 .../spark/structure/io/SparkContextStorage.java    | 107 +++++++++---
 .../structure/io/AbstractIoRegistryCheck.java      |   7 +-
 .../hadoop/structure/io/AbstractStorageCheck.java  |  48 +++---
 .../structure/io/FileSystemStorageCheck.java       |  24 ++-
 .../spark/process/computer/LocalPropertyTest.java  |   2 +-
 .../SparkHadoopGraphGryoSerializerProvider.java    |   1 +
 .../gremlin/spark/structure/SparkTest.java         |   3 +-
 .../structure/io/SparkContextStorageCheck.java     |  26 +--
 .../io/gryo/GryoSerializerIntegrateTest.java       |   4 +-
 .../gremlin/tinkergraph/TinkerGraphProvider.java   |   4 +-
 .../tinkergraph/TinkerGraphUUIDProvider.java       |   4 +-
 .../structure/IoDataGenerationTest.java            | 127 +++++++-------
 .../tinkergraph/structure/TinkerGraphTest.java     |  13 +-
 71 files changed, 1033 insertions(+), 737 deletions(-)
 create mode 100644 gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/util/CoreTestHelper.java
 copy gremlin-driver/src/test/java/org/apache/tinkerpop/gremlin/driver/TokensTest.java => gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/AssertHelper.java (77%)
 delete mode 100644 gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/TestHelper.java
 create mode 100644 hadoop-gremlin/README.md


[tinkerpop] 01/01: Merge branch 'tp34'

Posted by sp...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit a2dfcd0f8174ce59d617efd47b8b596797bcd2b8
Merge: 4240b35 ef941e3
Author: stephen <sp...@gmail.com>
AuthorDate: Fri Nov 8 07:23:36 2019 -0500

    Merge branch 'tp34'

 docs/preprocessor/install-plugins.groovy           |   2 +-
 gremlin-archetype/pom.xml                          |   1 +
 .../tinkerpop/gremlin/console/ConsoleFs.groovy     |   8 +-
 .../console/commands/UninstallCommand.groovy       |  12 +-
 .../jsr223/DriverRemoteAcceptorIntegrateTest.java  |  25 +--
 .../console/jsr223/DriverRemoteAcceptorTest.java   |   4 +-
 gremlin-core/pom.xml                               |   6 +
 .../tinkerpop/gremlin/structure/io/Storage.java    | 101 +++++++++--
 .../tinkerpop/gremlin/util/CoreTestHelper.java     | 188 +++++++++++++++++++++
 .../SerializerTest.java => AssertHelper.java}      |  27 +--
 .../org/apache/tinkerpop/gremlin/TestHelper.java   | 147 ----------------
 .../jsr223/DefaultScriptCustomizerTest.java        |   6 +-
 .../gremlin/jsr223/ScriptEngineCacheTest.java      |   8 +-
 .../jsr223/ScriptFileGremlinPluginTest.java        |  11 +-
 .../gremlin/process/traversal/OrderTest.java       |   3 +-
 .../optimization/EarlyLimitStrategyTest.java       |   4 +-
 .../optimization/OrderLimitStrategyTest.java       |   2 +-
 .../StandardVerificationStrategyTest.java          |   6 +-
 .../gremlin/structure/io/GraphMigratorTest.java    |   4 +-
 .../tinkerpop/gremlin/structure/io/IoCoreTest.java |   4 +-
 .../structure/io/graphson/GraphSONTokensTest.java  |   4 +-
 .../structure/io/graphson/GraphSONUtilTest.java    |   4 +-
 .../gremlin/structure/util/ComparatorsTest.java    |   4 +-
 .../gremlin/structure/util/ElementHelperTest.java  |   4 +-
 .../gremlin/structure/util/GraphFactoryTest.java   |  32 ++--
 .../structure/util/GraphVariableHelperTest.java    |   4 +-
 .../gremlin/structure/util/StringFactoryTest.java  |   4 +-
 .../apache/tinkerpop/gremlin/util/GremlinTest.java |   4 +-
 .../tinkerpop/gremlin/util/SerializerTest.java     |   4 +-
 .../tinkerpop/gremlin/util/TimeUtilTest.java       |   4 +-
 .../gremlin/util/function/FunctionUtilsTest.java   |   5 +-
 .../gremlin/util/iterator/IteratorUtilsTest.java   |   4 +-
 .../tinkerpop/gremlin/util/tools/MultiMapTest.java |   5 +-
 .../gremlin/groovy/util/DependencyGrabber.groovy   |  74 ++++----
 .../gremlin/groovy/engine/GremlinExecutorTest.java |   4 +-
 .../GremlinGroovyScriptEngineFileSandboxTest.java  |   5 +-
 .../util/DependencyGrabberIntegrateTest.java       |  76 +++++----
 .../gremlin/util/TinkerGraphProvider.java          |   4 +-
 .../gremlin/server/util/GremlinServerInstall.java  |   7 +-
 .../gremlin/server/GremlinDriverIntegrateTest.java |   6 +-
 .../tinkerpop/gremlin/server/ServerTestHelper.java |  57 +++++--
 .../gremlin/util/Log4jRecordingAppender.java       |  17 +-
 .../gremlin/AbstractFileGraphProvider.java         |  12 +-
 .../apache/tinkerpop/gremlin/GraphProvider.java    |   3 +-
 .../org/apache/tinkerpop/gremlin/TestHelper.java   | 142 ++--------------
 .../gremlin/jsr223/BindingsScriptEngineTest.java   |   3 +-
 .../gremlin/structure/io/IoGraphTest.java          |   8 +-
 .../tinkerpop/gremlin/structure/io/IoTest.java     |  31 ++--
 hadoop-gremlin/README.md                           |  52 ++++++
 hadoop-gremlin/pom.xml                             |  29 ++++
 .../apache/tinkerpop/gremlin/hadoop/Constants.java |  22 +++
 .../computer/AbstractHadoopGraphComputer.java      |  12 +-
 .../hadoop/structure/io/FileSystemStorage.java     | 101 ++++++-----
 .../hadoop/structure/io/InputOutputHelper.java     |   2 +-
 .../computer/AbstractHadoopGraphComputerTest.java  |  59 ++++---
 .../structure/io/RecordReaderWriterTest.java       |   5 +-
 pom.xml                                            |  18 +-
 .../tinkerpop/gremlin/spark/structure/Spark.java   |   7 +-
 .../spark/structure/io/SparkContextStorage.java    | 107 +++++++++---
 .../structure/io/AbstractIoRegistryCheck.java      |   7 +-
 .../hadoop/structure/io/AbstractStorageCheck.java  |  48 +++---
 .../structure/io/FileSystemStorageCheck.java       |  24 ++-
 .../spark/process/computer/LocalPropertyTest.java  |   2 +-
 .../SparkHadoopGraphGryoSerializerProvider.java    |   1 +
 .../gremlin/spark/structure/SparkTest.java         |   3 +-
 .../structure/io/SparkContextStorageCheck.java     |  26 +--
 .../io/gryo/GryoSerializerIntegrateTest.java       |   4 +-
 .../gremlin/tinkergraph/TinkerGraphProvider.java   |   4 +-
 .../tinkergraph/TinkerGraphUUIDProvider.java       |   4 +-
 .../structure/IoDataGenerationTest.java            | 127 +++++++-------
 .../tinkergraph/structure/TinkerGraphTest.java     |  13 +-
 71 files changed, 1036 insertions(+), 741 deletions(-)

diff --cc gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/structure/util/GraphFactoryTest.java
index c146f72,514b28c..5fc6951
--- a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/structure/util/GraphFactoryTest.java
+++ b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/structure/util/GraphFactoryTest.java
@@@ -18,9 -18,9 +18,9 @@@
   */
  package org.apache.tinkerpop.gremlin.structure.util;
  
 -import org.apache.commons.configuration.BaseConfiguration;
 -import org.apache.commons.configuration.Configuration;
 +import org.apache.commons.configuration2.BaseConfiguration;
 +import org.apache.commons.configuration2.Configuration;
- import org.apache.tinkerpop.gremlin.TestHelper;
+ import org.apache.tinkerpop.gremlin.AssertHelper;
  import org.apache.tinkerpop.gremlin.process.computer.GraphComputer;
  import org.apache.tinkerpop.gremlin.structure.Edge;
  import org.apache.tinkerpop.gremlin.structure.Graph;
diff --cc gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/AbstractFileGraphProvider.java
index 636ea3d,18e9a0d..cedbf38
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/AbstractFileGraphProvider.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/AbstractFileGraphProvider.java
@@@ -18,8 -18,9 +18,9 @@@
   */
  package org.apache.tinkerpop.gremlin;
  
 -import org.apache.commons.configuration.Configuration;
 +import org.apache.commons.configuration2.Configuration;
  import org.apache.tinkerpop.gremlin.structure.Graph;
+ import org.apache.tinkerpop.gremlin.structure.io.Storage;
  import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONResourceAccess;
  import org.apache.tinkerpop.gremlin.structure.io.gryo.GryoResourceAccess;
  import org.apache.tinkerpop.gremlin.structure.io.script.ScriptResourceAccess;