[jira] [Commented] (TINKERPOP-1301) Provide Javadoc for ScriptInput/OutputFormat's

[ https://issues.apache.org/jira/browse/TINKERPOP-1301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15315071#comment-15315071 ] ASF GitHub Bot commented on TINKERPOP-1301: --- Github user asfgit closed the pull request at:

[GitHub] incubator-tinkerpop pull request #327: TINKERPOP-1301 Provide Javadoc for Sc...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-tinkerpop/pull/327 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[jira] [Commented] (TINKERPOP-1317) IoGraphTest throws error: URI is not hierarchical

[ https://issues.apache.org/jira/browse/TINKERPOP-1317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314949#comment-15314949 ] ASF GitHub Bot commented on TINKERPOP-1317: --- Github user twilmes commented on a diff in the

[GitHub] incubator-tinkerpop pull request #328: TINKERPOP-1317: use graph class as ro...

Github user twilmes commented on a diff in the pull request: https://github.com/apache/incubator-tinkerpop/pull/328#discussion_r65783485 --- Diff: docs/src/dev/provider/index.asciidoc --- @@ -505,6 +505,11 @@ off for the test suite (the Maven SureFire Plugin is configured this way

[GitHub] incubator-tinkerpop issue #328: TINKERPOP-1317: use graph class as root clas...

Github user analytically commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/328 Genius PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] incubator-tinkerpop issue #328: TINKERPOP-1317: use graph class as root clas...

Github user dkuppitz commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/328 VOTE: +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[jira] [Updated] (TINKERPOP-1325) Query results with lambda step not serializable in OLAP

[ https://issues.apache.org/jira/browse/TINKERPOP-1325?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette updated TINKERPOP-1325: Affects Version/s: (was: 3.2.1) 3.2.0-incubating

[jira] [Updated] (TINKERPOP-1325) Query results with lambda step not serializable in OLAP

[ https://issues.apache.org/jira/browse/TINKERPOP-1325?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sergey Gmyzov updated TINKERPOP-1325: - Description: Query with lambda step fails in OLAP mode: {code} gremlin> :remote

[jira] [Created] (TINKERPOP-1325) Query results with lambda step not serializable in OLAP

Sergey Gmyzov created TINKERPOP-1325: Summary: Query results with lambda step not serializable in OLAP Key: TINKERPOP-1325 URL: https://issues.apache.org/jira/browse/TINKERPOP-1325 Project:

[jira] [Commented] (TINKERPOP-1321) Loosen coupling between TinkerPop serialization logic and shaded Kryo

[ https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314682#comment-15314682 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user dalaro commented on the issue:

[GitHub] incubator-tinkerpop issue #325: TINKERPOP-1321 Introduce Kryo shim to suppor...

Github user dalaro commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/325 Additional notes from Marko: > okay, so as long as there is never a need for MyRegistrar, then can you please update your PR accordingly? > Finally, call it GryoRegistrar

[jira] [Commented] (TINKERPOP-1321) Loosen coupling between TinkerPop serialization logic and shaded Kryo

[ https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314655#comment-15314655 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user dalaro commented on the issue:

[GitHub] incubator-tinkerpop issue #325: TINKERPOP-1321 Introduce Kryo shim to suppor...

Github user okram commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/325 @dalaro -- what about it doesn't work? Meaning, is your problem just a "unregistered class" issue? If so, we just register it. If I have a "bad" serializer for one of the registered

[jira] [Commented] (TINKERPOP-1321) Loosen coupling between TinkerPop serialization logic and shaded Kryo

[ https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314508#comment-15314508 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user dalaro commented on the issue:

[GitHub] incubator-tinkerpop issue #325: TINKERPOP-1321 Introduce Kryo shim to suppor...

Github user dalaro commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/325 The problem is that GryoSerializer doesn't actually work. This is not the first time it has shown behavior on Spark or scala runtime classes which diverges from the behavior of Spark's

[jira] [Commented] (TINKERPOP-1321) Loosen coupling between TinkerPop serialization logic and shaded Kryo

[ https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314503#comment-15314503 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user okram commented on the issue:

[GitHub] incubator-tinkerpop issue #325: TINKERPOP-1321 Introduce Kryo shim to suppor...

Github user okram commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/325 So yea... sorta feels like we are where we were before in a way. And yea, all that Input, Output, Kryo stuff is what causes all the problems. So now I'm wondering, why not just

[jira] [Commented] (TINKERPOP-1321) Loosen coupling between TinkerPop serialization logic and shaded Kryo

[ https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314456#comment-15314456 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user dalaro commented on the issue:

[jira] [Commented] (TINKERPOP-1318) java.lang.NoSuchMethodError: org/hamcrest/Matcher.describeMismatch

[ https://issues.apache.org/jira/browse/TINKERPOP-1318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314431#comment-15314431 ] ASF GitHub Bot commented on TINKERPOP-1318: --- Github user pluradj commented on the issue:

[GitHub] incubator-tinkerpop issue #329: TINKERPOP-1318: use mockito-core instead of ...

Github user pluradj commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/329 Yes, I ran integration tests with `-DincludeNeo4j` --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[jira] [Commented] (TINKERPOP-1319) several FeatureRequirement annotations are incorrect in gremlin-test

[ https://issues.apache.org/jira/browse/TINKERPOP-1319?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314427#comment-15314427 ] ASF GitHub Bot commented on TINKERPOP-1319: --- GitHub user pluradj opened a pull request:

[GitHub] incubator-tinkerpop pull request #330: several FeatureRequirement annotation...

GitHub user pluradj opened a pull request: https://github.com/apache/incubator-tinkerpop/pull/330 several FeatureRequirement annotations are incorrect https://issues.apache.org/jira/browse/TINKERPOP-1319 You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-tinkerpop issue #329: use mockito-core instead of mockito-all to a...

Github user okram commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/329 Do integration tests pass? ... that will be all I care about. @spmallette will have deeper thoughts on the matter as he uses Mockito. --- If your project is set up for it, you can reply

[jira] [Commented] (TINKERPOP-1318) java.lang.NoSuchMethodError: org/hamcrest/Matcher.describeMismatch

[ https://issues.apache.org/jira/browse/TINKERPOP-1318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314426#comment-15314426 ] ASF GitHub Bot commented on TINKERPOP-1318: --- GitHub user pluradj opened a pull request:

[jira] [Commented] (TINKERPOP-1321) Loosen coupling between TinkerPop serialization logic and shaded Kryo

[ https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314387#comment-15314387 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user okram commented on the issue:

[GitHub] incubator-tinkerpop issue #327: TINKERPOP-1301 Provide Javadoc for ScriptInp...

Github user lewismc commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/327 @spmallette LOL np --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[jira] [Commented] (TINKERPOP-1301) Provide Javadoc for ScriptInput/OutputFormat's

[ https://issues.apache.org/jira/browse/TINKERPOP-1301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314371#comment-15314371 ] ASF GitHub Bot commented on TINKERPOP-1301: --- Github user pluradj commented on the issue:

[GitHub] incubator-tinkerpop issue #327: TINKERPOP-1301 Provide Javadoc for ScriptInp...

Github user pluradj commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/327 VOTE +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[jira] [Commented] (TINKERPOP-1301) Provide Javadoc for ScriptInput/OutputFormat's

[ https://issues.apache.org/jira/browse/TINKERPOP-1301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15314306#comment-15314306 ] ASF GitHub Bot commented on TINKERPOP-1301: --- GitHub user lewismc opened a pull request:

[GitHub] incubator-tinkerpop issue #325: TINKERPOP-1321 Introduce Kryo shim to suppor...

Github user dalaro commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/325 @spmallette You're right about overrides being the problem. The last thing I did before opening this PR was to rebase on latest master, when I encountered conflicts solely

[GitHub] incubator-tinkerpop issue #325: TINKERPOP-1321 Introduce Kryo shim to suppor...

Github user okram commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/325 @dalaro -- you mention: ``` spark.serializer=KryoSerializer spark.kryo.registrator=com.tinkerpop.something.TinkerPopRegistrator ``` I think we need:

[jira] [Commented] (TINKERPOP-1196) Calls to Result.one() might block indefinitely

[ https://issues.apache.org/jira/browse/TINKERPOP-1196?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15313981#comment-15313981 ] ASF GitHub Bot commented on TINKERPOP-1196: --- Github user dkuppitz commented on the issue:

[jira] [Commented] (TINKERPOP-1321) Loosen coupling between TinkerPop serialization logic and shaded Kryo

[ https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15313973#comment-15313973 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user spmallette commented on the issue:

[GitHub] incubator-tinkerpop issue #325: TINKERPOP-1321 Introduce Kryo shim to suppor...

Github user spmallette commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/325 I've confirmed that this test: ```text GremlinResultSetIntegrateTest.shouldHandleVertexResultWithLiteSerialization:174 ? Execution ``` is only failing after