[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15317589#comment-15317589
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user dalaro commented on
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.
---
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15317578#comment-15317578
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user okram commented on
Github user okram commented on the issue:
https://github.com/apache/incubator-tinkerpop/pull/325
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, o
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15317312#comment-15317312
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user spmallette commente
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,
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15317304#comment-15317304
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user okram commented on
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 about
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15317291#comment-15317291
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user spmallette commente
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
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15317279#comment-15317279
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user dalaro commented on
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.
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15316836#comment-15316836
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user okram commented on
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 to
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15316810#comment-15316810
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user okram commented on
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 excepti
Github user okram commented on the issue:
https://github.com/apache/incubator-tinkerpop/pull/325
I think some things need to be fixed up.
1. `TinkerPopKryoRegistrator` should be called `GryoRegistrator`.
2. I think we need a `GryoPoolKryoShimService` in core that
non-Spark
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15316622#comment-15316622
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user okram commented on
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 chair it is my responsib
[
https://issues.apache.org/jira/browse/TINKERPOP-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15316514#comment-15316514
]
ASF GitHub Bot commented on TINKERPOP-1321:
---
Github user okram commented on
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:
GroovyProgramTest$Traversals>ProgramTest.g_V_outXcreatedX_aggregateXxX_byXlangX_groupCount_programXTest
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 thinking
> that we could sta
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 branding perspective
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.
>
> Homepage, mailing lists do
Github user spmallette commented on the issue:
https://github.com/apache/incubator-tinkerpop/pull/315
@analytically I see that you made some updates. I think you might need to
rebase against tp31 though to resolve conflicts. Other than that, is this ready
for review at this point?
-
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 fe
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 projec
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-tinkerpop/pull/326
---
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 fe
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
Github user dalaro commented on the issue:
https://github.com/apache/incubator-tinkerpop/pull/325
I just pushed some changes that I hacked together this weekend. The key
additions are:
* `TinkerPopKryoRegistrator`, which I extracted from my app, and which acts
as a `spark.kr
30 matches
Mail list logo
