Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1173

If no Serializer is provided in Configuration, use GryoSerializer by default (Spark)

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.1.0-incubating
    • 3.2.0-incubating
    • process
    • None

    Description

      Attachments

        Activity

          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/incubator-tinkerpop/pull/255

          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/incubator-tinkerpop/pull/255
          githubbot ASF GitHub Bot added a comment -

          Github user dkuppitz commented on the pull request:

          https://github.com/apache/incubator-tinkerpop/pull/255#issuecomment-194571252

          VOTE: +1

          githubbot ASF GitHub Bot added a comment - Github user dkuppitz commented on the pull request: https://github.com/apache/incubator-tinkerpop/pull/255#issuecomment-194571252 VOTE: +1
          githubbot ASF GitHub Bot added a comment -

          Github user spmallette commented on the pull request:

          https://github.com/apache/incubator-tinkerpop/pull/255#issuecomment-194560408

          builds/tests nicely for me: VOTE +1

          githubbot ASF GitHub Bot added a comment - Github user spmallette commented on the pull request: https://github.com/apache/incubator-tinkerpop/pull/255#issuecomment-194560408 builds/tests nicely for me: VOTE +1
          githubbot ASF GitHub Bot added a comment -

          GitHub user okram opened a pull request:

          https://github.com/apache/incubator-tinkerpop/pull/255

          TINKERPOP-1173: If no Serializer is provided in Configuration, use GryoSerializer by default (Spark)

          https://issues.apache.org/jira/browse/TINKERPOP-1173

          If the user does not specify a serializer, then default to using `GryoSerializer` instead of Spark's default of `JavaSerializer`. Updated a test case to verify the behavior.

          ```
          [INFO] ------------------------------------------------------------------------
          [INFO] BUILD SUCCESS
          [INFO] ------------------------------------------------------------------------
          [INFO] Total time: 14:00.006s
          [INFO] Finished at: Wed Mar 09 11:00:50 MST 2016
          [INFO] Final Memory: 133M/806M
          [INFO] ------------------------------------------------------------------------
          ```

          VOTE +1.

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1173

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/incubator-tinkerpop/pull/255.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #255


          commit 2e04cf8655799e5a75f8a90811cb0cf3ba856031
          Author: Marko A. Rodriguez <okrammarko@gmail.com>
          Date: 2016-03-09T17:47:38Z

          If no spark.serializer is provided, then use GryoSerializer by default – not JavaSerialzer (as Spark defaults to).


          githubbot ASF GitHub Bot added a comment - GitHub user okram opened a pull request: https://github.com/apache/incubator-tinkerpop/pull/255 TINKERPOP-1173 : If no Serializer is provided in Configuration, use GryoSerializer by default (Spark) https://issues.apache.org/jira/browse/TINKERPOP-1173 If the user does not specify a serializer, then default to using `GryoSerializer` instead of Spark's default of `JavaSerializer`. Updated a test case to verify the behavior. ``` [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 14:00.006s [INFO] Finished at: Wed Mar 09 11:00:50 MST 2016 [INFO] Final Memory: 133M/806M [INFO] ------------------------------------------------------------------------ ``` VOTE +1. You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1173 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-tinkerpop/pull/255.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #255 commit 2e04cf8655799e5a75f8a90811cb0cf3ba856031 Author: Marko A. Rodriguez <okrammarko@gmail.com> Date: 2016-03-09T17:47:38Z If no spark.serializer is provided, then use GryoSerializer by default – not JavaSerialzer (as Spark defaults to).
          dkuppitz Daniel Kuppitz added a comment -

          Smart idea .

          dkuppitz Daniel Kuppitz added a comment - Smart idea .

          People

            okram Marko A. Rodriguez
            okram Marko A. Rodriguez
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: