travis failures

2016-02-10 Thread Stephen Mallette
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] incubator-tinkerpop pull request: attempt to fix Travis integratio...

2016-02-10 Thread dkuppitz
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

Re: travis failures

2016-02-10 Thread Daniel Kuppitz
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] incubator-tinkerpop pull request: TINKERPOP-1125 Use the existing ...

2016-02-10 Thread spmallette
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] incubator-tinkerpop pull request: TINKERPOP-1126

2016-02-10 Thread spmallette
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

[jira] [Commented] (TINKERPOP-1126) A single Host spawns many reconnect tasks

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

[jira] [Commented] (TINKERPOP-1125) RoundRobin load balancing always uses the second Host when size = 2

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

Re: travis failures

2016-02-10 Thread Stephen Mallette
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 >

[jira] [Updated] (TINKERPOP-1123) Execution time in console output

2016-02-10 Thread stephen mallette (JIRA)
[ 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 > --

[jira] [Updated] (TINKERPOP-1123) Execution time in console output

2016-02-10 Thread stephen mallette (JIRA)
[ 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 > ---

Re: travis failures

2016-02-10 Thread Daniel Kuppitz
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

Re: travis failures

2016-02-10 Thread Daniel Kuppitz
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

Re: travis failures

2016-02-10 Thread Stephen Mallette
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

[jira] [Updated] (TINKERPOP-1137) Deprecate ScriptElementFactory and make star graph globally available

2016-02-10 Thread Daniel Kuppitz (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread pluradj
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

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

Re: [VOTE] TinkerPop 3.1.1-incubating Release

2016-02-10 Thread Bryn Cooke
+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

[DISCUSS] Artifact production in test phase

2016-02-10 Thread Stephen Mallette
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

Re: [VOTE] TinkerPop 3.1.1-incubating Release

2016-02-10 Thread Bob B
+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

[jira] [Updated] (TINKERPOP-1039) Enable auto-commit for session'd requests.

2016-02-10 Thread stephen mallette (JIRA)
[ 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 >

Re: [DISCUSS] Artifact production in test phase

2016-02-10 Thread Marko Rodriguez
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

Re: [VOTE] TinkerPop 3.1.1-incubating Release

2016-02-10 Thread Daniel Kuppitz
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

Re: [DISCUSS] Artifact production in test phase

2016-02-10 Thread Stephen Mallette
> 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

Re: [DISCUSS] Artifact production in test phase

2016-02-10 Thread Marko Rodriguez
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

Re: [jira] [Created] (TINKERPOP-1136) GroupStep OLAP does not combine, only reduces.

2016-02-10 Thread Ran Magen
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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread velo
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:

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread pluradj
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

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread velo
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

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread pluradj
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

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread velo
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

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-916 Provide more robus...

2016-02-10 Thread dkuppitz
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

[jira] [Commented] (TINKERPOP-916) Develop a better "simple" driver for testing and example purposes

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1146 Use GraphProvider...

2016-02-10 Thread spmallette
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()`

[jira] [Commented] (TINKERPOP-1146) IoTest are not clearing the db after the test run

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: clear graph after IoTest.shouldR...

2016-02-10 Thread spmallette
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] incubator-tinkerpop pull request: TINKERPOP-916 Provide more robus...

2016-02-10 Thread asfgit
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

[jira] [Commented] (TINKERPOP-916) Develop a better "simple" driver for testing and example purposes

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

[jira] [Created] (TINKERPOP-1147) Add serialization for TraversalExplanation

2016-02-10 Thread Bob Briody (JIRA)
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

[jira] [Updated] (TINKERPOP-1147) Add serialization for TraversalExplanation

2016-02-10 Thread stephen mallette (JIRA)
[ 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

[jira] [Updated] (TINKERPOP-1147) Add serialization for TraversalExplanation

2016-02-10 Thread stephen mallette (JIRA)
[ 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 > -

Re: [VOTE] TinkerPop 3.1.1-incubating Release

2016-02-10 Thread Jean-Baptiste Musso
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

[jira] [Commented] (TINKERPOP-1137) Deprecate ScriptElementFactory and make star graph globally available

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1137 Deprecate ScriptE...

2016-02-10 Thread dkuppitz
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] incubator-tinkerpop pull request: TINKERPOP-1137 Deprecate ScriptE...

2016-02-10 Thread okram
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

[jira] [Commented] (TINKERPOP-1137) Deprecate ScriptElementFactory and make star graph globally available

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1137 Deprecate ScriptE...

2016-02-10 Thread dkuppitz
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

[jira] [Commented] (TINKERPOP-1137) Deprecate ScriptElementFactory and make star graph globally available

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

Re: [VOTE] TinkerPop 3.1.1-incubating Release

2016-02-10 Thread David Brown
+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] incubator-tinkerpop pull request: TINKERPOP-1137 Deprecate ScriptE...

2016-02-10 Thread okram
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

[jira] [Commented] (TINKERPOP-1137) Deprecate ScriptElementFactory and make star graph globally available

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

[DISCUSS] CHANGELOG and update.asciidoc after VOTE?

2016-02-10 Thread Marko Rodriguez
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

Re: [jira] [Created] (TINKERPOP-1136) GroupStep OLAP does not combine, only reduces.

2016-02-10 Thread Marko Rodriguez
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/

[jira] [Closed] (TINKERPOP-1147) Add serialization for TraversalExplanation

2016-02-10 Thread Marko A. Rodriguez (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-971: TraversalSource s...

2016-02-10 Thread okram
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

[jira] [Commented] (TINKERPOP-971) TraversalSource should be fluent like GraphComputer

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

Re: Enable CI build for the project

2016-02-10 Thread Marvin Froeder
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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread velo
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

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

Re: [jira] [Created] (TINKERPOP-1136) GroupStep OLAP does not combine, only reduces.

2016-02-10 Thread Ran Magen
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] incubator-tinkerpop pull request: TINKERPOP-1041

2016-02-10 Thread velo
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

[jira] [Commented] (TINKERPOP-1041) StructureStandardTestSuite has file I/O issues on Windows

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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

Re: [DISCUSS] CHANGELOG and update.asciidoc after VOTE?

2016-02-10 Thread Daniel Kuppitz
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'

[jira] [Commented] (TINKERPOP-971) TraversalSource should be fluent like GraphComputer

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-971: TraversalSource s...

2016-02-10 Thread dkuppitz
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] incubator-tinkerpop pull request: TINKERPOP-1146 Use GraphProvider...

2016-02-10 Thread twilmes
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

[jira] [Commented] (TINKERPOP-1146) IoTest are not clearing the db after the test run

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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] incubator-tinkerpop pull request: TINKERPOP-1137 Deprecate ScriptE...

2016-02-10 Thread twilmes
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

[jira] [Commented] (TINKERPOP-1137) Deprecate ScriptElementFactory and make star graph globally available

2016-02-10 Thread ASF GitHub Bot (JIRA)
[ 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