Anyone want to be a hero? Tell us why travis fails every build lately. all
fails look like this:
https://travis-ci.org/apache/incubator-tinkerpop/builds/108127093
The error at the bottom says:
[ERROR] Failed to execute goal
org.apache.maven.plugins:maven-surefire-plugin:2.17:test (default-test)
GitHub user dkuppitz opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/222
attempt to fix Travis integration
Ignore this PR, just trying to fix Travis.
http://stackoverflow.com/questions/23260057/the-forked-vm-terminated-without-saying-properly-goodb
Trying...
https://github.com/apache/incubator-tinkerpop/pull/222
On Wed, Feb 10, 2016 at 12:37 PM, Stephen Mallette
wrote:
> Anyone want to be a hero? Tell us why travis fails every build lately. all
> fails look like this:
>
> https://travis-ci.org/apache/incubator-tinkerpop/builds/108127093
>
Github user spmallette commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/211#issuecomment-182363924
I've merged this PR to a TinkerPop branch called `TINKERPOP-1125-to-1127`
for internal code review. it covers this PR and #213
That branch l
Github user spmallette commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/213#issuecomment-182364134
I've merged this PR to a TinkerPop branch called `TINKERPOP-1125-to-1127`
for internal code review. it covers this PR and #211 - please see the VOTE
t
[
https://issues.apache.org/jira/browse/TINKERPOP-1126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15140740#comment-15140740
]
ASF GitHub Bot commented on TINKERPOP-1126:
---
Github user spmallette commente
[
https://issues.apache.org/jira/browse/TINKERPOP-1125?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15140738#comment-15140738
]
ASF GitHub Bot commented on TINKERPOP-1125:
---
Github user spmallette commente
hmmm - permgen - that setting shouldn't have any affect in java 8 though:
https://dzone.com/articles/java-8-permgen-metaspace
i'd try anything though at this point...
On Wed, Feb 10, 2016 at 8:00 AM, Daniel Kuppitz wrote:
> Trying...
>
> https://github.com/apache/incubator-tinkerpop/pull/222
>
[
https://issues.apache.org/jira/browse/TINKERPOP-1123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette updated TINKERPOP-1123:
Issue Type: Improvement (was: Wish)
> Execution time in console output
> --
[
https://issues.apache.org/jira/browse/TINKERPOP-1123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette updated TINKERPOP-1123:
Affects Version/s: 3.1.0-incubating
> Execution time in console output
> ---
daniel@cube /projects/apache/incubator-tinkerpop (travis) $ grep -R
'System.exit' * | grep test | grep -v Binary
gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/jsr223/GremlinGroovyScriptEngineFileSandboxTest.java:
engine.eval("System.exit(0)");
gremlin-server/src/test/java/or
Hm, nope, even without the test cases the call System.exit(), Travis is
still not happy.
On Wed, Feb 10, 2016 at 2:20 PM, Daniel Kuppitz wrote:
> daniel@cube /projects/apache/incubator-tinkerpop (travis) $ grep -R
> 'System.exit' * | grep test | grep -v Binary
> gremlin-groovy-test/src/main/java
Yeah - i searched for System.exit(0) stuff but didn't see anything that
would cause the problem.
On Wed, Feb 10, 2016 at 8:53 AM, Daniel Kuppitz wrote:
> Hm, nope, even without the test cases the call System.exit(), Travis is
> still not happy.
>
> On Wed, Feb 10, 2016 at 2:20 PM, Daniel Kuppitz
[
https://issues.apache.org/jira/browse/TINKERPOP-1137?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Daniel Kuppitz updated TINKERPOP-1137:
--
Summary: Deprecate ScriptElementFactory and make star graph globally
available (was
Github user pluradj commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182410088
I'm running into some test failures when I do `mvn clean install` on
Windows. I built on Linux and Mac without running into this.
```
Tests run
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15140932#comment-15140932
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user pluradj commented o
+1
Downloaded and built src distribution
Used the gremlin console to fire off some queries.
Cheers,
Bryn
On 08/02/16 15:00, Stephen Mallette wrote:
Hello, The release artifacts can be found at this location:
https://dist.apache.org/repos/dist/dev/incubator/tinkerpop/3.1.1-incubating/
The sourc
I don't think we should create artifacts from test jars. I guess
hadoop-gremlin does that now and it prevents you from doing simple stuff
like:
mvn clean install -DskipTests
if it's the first time you're building a particular version (the artifact
can't be found)
[ERROR] Failed to execute goal
+1
Ran validation script w/ tests enabled.
uname -a
Linux rhino 3.13.0-77-generic #121-Ubuntu SMP Wed Jan 20 10:50:42 UTC 2016
x86_64 x86_64 x86_64 GNU/Linux
Validating binary distributions
* downloading Apache Gremlin Console
(apache-gremlin-console-3.1.1-incubating-bin.zip)... OK
* validatin
[
https://issues.apache.org/jira/browse/TINKERPOP-1039?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette updated TINKERPOP-1039:
Fix Version/s: (was: 3.2.0-incubating)
3.1.2-incubating
>
Hi,
The reason I did this was because there are test classes shared between Hadoop,
Giraph, and Spark. We could go the route of creating a hadoop-gremlin-test
package, but dah… I almost think we should get rid of gremlin-groovy-test and
gremlin-test and use the model in Hadoop.
Finally, I thi
All good.
Validating binary distributions
* downloading Apache Gremlin Console
(apache-gremlin-console-3.1.1-incubating-bin.zip)... OK
* validating signatures and checksums ...
* PGP signature ... OK
* MD5 checksum ... OK
* SHA1 chacksum ... OK
* unzipping Apache Gremlin Console ... OK
* va
> Finally, I think "mvn clean install -Dmaven.test.skip=true" works fine.
i don't think either works. that was what was in validate-distribution.sh
and it failed for folks.
something generally seems weird/wrong to me about generating an artifact in
a test phase. doesn't feel right.
On Wed, Feb
I hear ya, but hadoop-gremlin-test/ sucks…"yet another module."
?,
Marko.
http://markorodriguez.com
On Feb 10, 2016, at 9:32 AM, Stephen Mallette wrote:
>> Finally, I think "mvn clean install -Dmaven.test.skip=true" works fine.
>
> i don't think either works. that was what was in validate-di
I'm not sure if its relevant for this ticket, but I have another use-case
for combiners in an OLTP use-case.
In Unipop (to which I'll hopefully have documentation very soon and be able
to share with you guys) we have GroupStep optimizations. But since we can
have results for the same step coming fr
Github user velo commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182506783
Yes, I was not able to fix this errors.
But if you run either master or tp311, more will fail
On Thu, 11 Feb 2016 04:01 Jason Plurad wrote:
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141330#comment-15141330
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user velo commented on t
Github user pluradj commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182512844
You're right, fair enough.
Did you pull the TP3 codebase into Eclipse and run the tests within it?
The code changes look good to me.
---
If your pr
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141379#comment-15141379
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user pluradj commented o
Github user velo commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182515486
Yes, they also fail on eclipse
On Thu, 11 Feb 2016 07:18 Jason Plurad wrote:
> You're right, fair enough.
> Did you pull the TP3 codebase
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141391#comment-15141391
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user velo commented on t
Github user pluradj commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182516098
I was referring to the testcases from the JIRA... do those now work in
Eclipse?
---
If your project is set up for it, you can reply to this email and have
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141393#comment-15141393
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user pluradj commented o
Github user velo commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182523091
Ow, i never tried to run all tests in eclipse...
On Thu, 11 Feb 2016 07:25 Jason Plurad wrote:
> I was referring to the testcases from the JI
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141420#comment-15141420
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user velo commented on t
Github user dkuppitz commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/216#issuecomment-182528791
* `mvn clean install`: passed
* integration tests: passed
Code looks good.
VOTE: +1
---
If your project is set up for it, you can re
[
https://issues.apache.org/jira/browse/TINKERPOP-916?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141443#comment-15141443
]
ASF GitHub Bot commented on TINKERPOP-916:
--
Github user dkuppitz commented on
GitHub user spmallette opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/223
TINKERPOP-1146 Use GraphProvider.clear() to auto-clear graphs created in
tests
https://issues.apache.org/jira/browse/TINKERPOP-1146
Was formerly just calling `Graph.close()`
[
https://issues.apache.org/jira/browse/TINKERPOP-1146?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141545#comment-15141545
]
ASF GitHub Bot commented on TINKERPOP-1146:
---
GitHub user spmallette opened a
Github user spmallette commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/221#issuecomment-182553248
@pietermartin thanks for raising this issue and for offering a fix. i have
a more complete solution to the problem on this #223 - can you please take a
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-tinkerpop/pull/216
---
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
[
https://issues.apache.org/jira/browse/TINKERPOP-916?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141560#comment-15141560
]
ASF GitHub Bot commented on TINKERPOP-916:
--
Github user asfgit closed the pull
Bob Briody created TINKERPOP-1147:
-
Summary: Add serialization for TraversalExplanation
Key: TINKERPOP-1147
URL: https://issues.apache.org/jira/browse/TINKERPOP-1147
Project: TinkerPop
Issue
[
https://issues.apache.org/jira/browse/TINKERPOP-1147?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette updated TINKERPOP-1147:
Issue Type: Improvement (was: Bug)
> Add serialization for TraversalExplanation
[
https://issues.apache.org/jira/browse/TINKERPOP-1147?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette updated TINKERPOP-1147:
Fix Version/s: 3.1.2-incubating
> Add serialization for TraversalExplanation
> -
On Mon, Feb 8, 2016 at 4:00 PM, Stephen Mallette wrote:
> Hello, The release artifacts can be found at this location:
> https://dist.apache.org/repos/dist/dev/incubator/tinkerpop/3.1.1-incubating/
> The source distribution is provided by:
> apache-tinkerpop-3.1.1-incubating-src.zip
> Two binary di
[
https://issues.apache.org/jira/browse/TINKERPOP-1137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141816#comment-15141816
]
ASF GitHub Bot commented on TINKERPOP-1137:
---
Github user dkuppitz commented
Github user dkuppitz commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/220#issuecomment-182604992
* `mvn clean install`: passed
* `bin/process-docs.sh`: passed
* integration tests: passed
VOTE: +1
---
If your project is set up for it,
Github user okram commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/220#issuecomment-182606650
This is looking much better. Given that this is backwards compatible (nice
Exception catching trick BTW), can you make at least one of the script.groovy
in t
[
https://issues.apache.org/jira/browse/TINKERPOP-1137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141828#comment-15141828
]
ASF GitHub Bot commented on TINKERPOP-1137:
---
Github user okram commented on
Github user dkuppitz commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/220#issuecomment-182614201
Reverted the changes in `grateful-dead-script-input.groovy`, so it's using
the old model. `script-input.groovy` uses the new model. Both scripts are used
[
https://issues.apache.org/jira/browse/TINKERPOP-1137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141851#comment-15141851
]
ASF GitHub Bot commented on TINKERPOP-1137:
---
Github user dkuppitz commented
+1 (non-binding)
gremlinclient and aiogremlin tests run
against apache-gremlin-server-3.1.1-incubating w/default conf settings
gremlinrestclient runs w/gremlin-server-rest-modern.yaml conf
Thanks for all the hard work!
Best,
Dave
On Wed, Feb 10, 2016 at 4:51 PM, Jean-Baptiste Musso
wrote:
>
Github user okram commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/220#issuecomment-182629585
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 f
[
https://issues.apache.org/jira/browse/TINKERPOP-1137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141910#comment-15141910
]
ASF GitHub Bot commented on TINKERPOP-1137:
---
Github user okram commented on
Hi,
I feel like we have lost data cause of merge conflicts with CHANGELOG in the
past. Instead of updating the CHANGELOG (and update.asciidoc) prior to VOTE, I
have been adding that information to the PR. For example, see this PR:
https://github.com/apache/incubator-tinkerpop/pull/210 … Then wh
Hi Ran,
GroupStep in OLTP does use "on the fly"-reduction.
https://github.com/apache/incubator-tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStep.java#L176-L182
https://github.com/apache/incubator-tinkerpop/
[
https://issues.apache.org/jira/browse/TINKERPOP-1147?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marko A. Rodriguez closed TINKERPOP-1147.
-
Resolution: Fixed
Assignee: Marko A. Rodriguez
> Add serialization for T
Github user okram commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/215#issuecomment-182636140
Note that when TINKERPOP-1140 is completed,
`traversal.getStrategies().onGraphComputer()` will be replaced by
`TraversalHelper.onGraphComputer(traversal)`. M
[
https://issues.apache.org/jira/browse/TINKERPOP-971?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141955#comment-15141955
]
ASF GitHub Bot commented on TINKERPOP-971:
--
Github user okram commented on the
I created a issue on apache infra:
https://issues.apache.org/jira/browse/INFRA-11249
May be the team could endorse what I asked for.
Marvin
On Thu, Feb 4, 2016 at 1:14 AM, Stephen Mallette
wrote:
> I've confirmed that Apache Infra doesn't have an account to appveyor so we
> can't turn this on
Github user velo commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182656725
@spmallette I just asked apache infra for it.
https://issues.apache.org/jira/browse/INFRA-11249
Hopefully we will get that working.
---
If your p
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15142035#comment-15142035
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user velo commented on t
Oh, that works perfectly. Awesome!
On Thu, 11 Feb 2016 at 01:15 Marko Rodriguez wrote:
> Hi Ran,
>
> GroupStep in OLTP does use "on the fly"-reduction.
>
>
> https://github.com/apache/incubator-tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/m
Github user velo commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/209#issuecomment-182671692
BTW, @pluradj any idea on what to do on the broken tests?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
[
https://issues.apache.org/jira/browse/TINKERPOP-1041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15142125#comment-15142125
]
ASF GitHub Bot commented on TINKERPOP-1041:
---
Github user velo commented on t
I've never had a problem solving merge conflicts in CHANGELOG. That's
usually pretty easy, since you most likely want to keep both blocks. I'd
say "VOTE, merge, update CHANGELOG" makes it a lot easier to forget to
update CHANGELOG. Furthermore, even if 2 or 3 people reviewed and voted on
the PR, I'
[
https://issues.apache.org/jira/browse/TINKERPOP-971?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15142192#comment-15142192
]
ASF GitHub Bot commented on TINKERPOP-971:
--
Github user dkuppitz commented on
Github user dkuppitz commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/215#issuecomment-182685148
* `mvn clean install`: passed
* integration tests (incl. Neo4j): passed
VOTE: +1
One negative note though: Strategies are mainly resps
Github user twilmes commented on a diff in the pull request:
https://github.com/apache/incubator-tinkerpop/pull/223#discussion_r52563479
--- Diff: docs/src/upgrade/release-3.1.x-incubating.asciidoc ---
@@ -22,6 +22,32 @@
image::https://raw.githubusercontent.com/apache/incubator-tin
[
https://issues.apache.org/jira/browse/TINKERPOP-1146?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15142235#comment-15142235
]
ASF GitHub Bot commented on TINKERPOP-1146:
---
Github user twilmes commented o
Github user twilmes commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/220#issuecomment-182709226
Looks good
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
[
https://issues.apache.org/jira/browse/TINKERPOP-1137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15142255#comment-15142255
]
ASF GitHub Bot commented on TINKERPOP-1137:
---
Github user twilmes commented o
73 matches
Mail list logo