[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=15317589#comment-15317589 ] 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 Just a reminder: we are voting on branch https://github.com/apache/incubator-tinkerpop/tree/TINKERPOP-1321. That subsumes the PR and adds a combination of fixes from Marko and me. ---

[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=15317578#comment-15317578 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user okram 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=15317312#comment-15317312 ] 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 Yeah - i know - it's there, but it's proliferating and becoming more and more OK which i think is "bad". i'll bring it up as a separate issue. --- If your project is set up for it,

[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=15317304#comment-15317304 ] 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 We use `Random` in numerous places to test various "versions" of a particular code block. It is possible to replace all possible choices with new `GraphProviders` but you are talking

[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=15317291#comment-15317291 ] 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 > Marko setup SparkHadoopGraphProvider to randomly choose either the old serialization code or the new stuff in e700363#diff-afffc6796745845d46e6f60ea3a992f9R91. IMHO that should be

[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=15317279#comment-15317279 ] 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 These questions involved a lot of back-and-forth that would have been slow to resolve on github comments, so and I discussed the failures and the code on a hangout earlier today.

[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=15316836#comment-15316836 ] 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, `mvn clean install` is happy w/o the `TravesralInterruptionComputerTest` (commented out). Running full integration tests now... --- If your project is set up for it, you can reply

[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=15316810#comment-15316810 ] 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 Okay. So 1 and 2 above are done, however, I had to `OPT_OUT` of `TraversalInterruptionComputerTest` as for some reason, the `GryoPoolKryoShimService` is not respecting interruption

[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=15316622#comment-15316622 ] ASF GitHub Bot commented on TINKERPOP-1321: --- Github user okram commented on the issue:

Re: [DISCUSS] ASF Board Draft Report - June 2016

This looks good to me. Thanks, Ted On Sat, Jun 4, 2016, 11:54 AM Stephen Mallette wrote: > As TinkerPop has graduated from incubator, we no longer need to produce > incubator reports. Instead, TinkerPop needs to now provide reports to the > ASF Board. Ultimately, as PMC

[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=15316514#comment-15316514 ] 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 Testing this morning, I get two test failures. ``` Tests in error:

Re: [DISCUSS] Thinking on Release

Yes, end of June-ish is best for me as I have few things on my plate the first half of this month. Thanks, Marko. http://markorodriguez.com > On Jun 6, 2016, at 4:46 AM, Stephen Mallette wrote: > > We didn't really discuss a date for release on this thread. I was

Re: [DISCUSS] Thinking on Release

We didn't really discuss a date for release on this thread. I was thinking that we could start looking at the week of July 4th as the target week for VOTE and nail down a date as we get closer. On Wed, May 25, 2016 at 2:42 PM, Jason Plurad wrote: > I'd think from a TinkerPop

Re: When does incubator-suffix get dropped?

Gruno has said that there were problems in infrastructure with the incubation "upgrade" process - like stuff was broken. That was most of the hangup as I understood it. On Sat, Jun 4, 2016 at 2:40 PM, Hadrian Zbarcea wrote: > As soon as the graduation is announced. > >

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

Github user asfgit closed the pull request at: https://github.com/apache/incubator-tinkerpop/pull/314 --- 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

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

Github user spmallette commented on the issue: https://github.com/apache/incubator-tinkerpop/pull/314 Replaced by #327 - going to close this one from our end. --- 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

[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 I saw the `shouldHandleMonthDay` test failure on CI and pushed a fix. I carelessly broke the read leg of that serializer while doing some tedious, boilerplate-y method signature changes