Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/725
I ended up using a local copy of hadoop 2.7.2 to complete my docker rebuild.
Anyway, 'Reference Documentation' is now a local link but everything else
still
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/725
Hmm.. even Reference Doc isn't local. Thought it was the first time I
looked at it.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/725
Ignore that. Somehow my branch was reset to master. Rebuilding...
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/725
This makes reviewing so much easier!
VOTE +1
If anyone is interested, these are the times (in seconds) for building each
doc:
15 /usr/src/tinkermem/docs/src/recipes/duplicate
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/725
@dkuppitz It was change 37a2471932229ae6a4c1f5e5f3f2d3d25bbf7fd4 that
introduced dotnet that required the need to rebuild the images.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/725
That's fine. Now I know what to look for ;-)
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/727
@boliza I created a JIRA issue. Please review it. Can you change the title
to include 'TINKERPOP-1796' and put the link
'https://issues.apache.org/jira/browse/TINKERPOP-179
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/727
Thanks.
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/709
Looks good.
VOTE +1
---
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/346
ResultSet.hasNext should be idempotent
Sorry I didn't create a ticket; JIRA is having problems.
Calling hasNext() multiple times would dequeue items.
You can merge this
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/346
Yes, I ran 'mvn clean install -DskipIntegrationTests=false -DincludeNeo4j'
and it passed.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/346
I merged your master into my fork. Commit
0787e46097cc09bb278ac1c787c4a50ece81cdb0
I didn't realize it would affect the pull request. Next time I'll create a
branch.
---
If yo
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/346
Normally that's what I would do but tried doing it the 'github way' -
https://help.github.com/articles/syncing-a-fork/
So let me know what you want me to do now. I
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/352
Using (have to ignore hadoop/spark) mvn clean install -pl
gremlin-shaded,gremlin-core,gremlin-test,gremlin-groovy,gremlin-groovy-test,tinkergraph-gremlin,neo4j-gremlin,gremlin-driver,gremlin
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/354
BUILD SUCCESS
---
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
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/352
Passed for me. I'll run it a few more times to be sure.
---
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 doe
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/352
5 consecutive passes
---
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 user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/351
Some suggestions:
1. A type is not a class. Call types 'type' or '@type' (not clear the @ is
necessary since it's in the metadata payload anyway); Not @class. (Oth
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/352
Can you provide the stacktrace (from the surefire report)? What OS, java
versions?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/352
@spmallette I think the
GremlinServerAuthOldIntegrateTest.shouldFailAuthenticateWithPlainTextNoCredentials()
should remove the assertion startsWith("Invalid name provided&qu
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/351
@newkek @spmallette Sorry, my context was only this thread. I agree with
you on all accounts.
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/356
TINKERPOP-1338 Bumped groovy to 2.4.7
No new dependencies introduced.
No other dependency versions changed.
Successful build and verify with:
mvn clean install -pl
gremlin
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/356
Fixed my Spark issue. Wanted to rebuild since giraph, hadoop, and spark
have some dependency on groovy.
Builds cleanly with mvn clean install && mvn verify -pl gremli
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/356
This PR does include #353 but not #354 . In the meantime, I'm setting up
docker. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/356
I rebased to include @okram 's fix in the jira ticket. 4.5 hours later,
"docker/build.sh -t -i -n" passed!
---
If your project is set up for it, you can reply to this emai
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/351
So I've caught up on the discussion and I'll offer some more food for
thought since I haven't seen any other ideas. Embedding metadata is neither
easy nor fun (not for me a
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/361
Java is brain damaged when comparing numbers. Don't blame JSON.
---
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 pr
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/362
Minor word and link cleanup
Fixed spelling and links.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop docs-edits
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/365
Tp31: Minor word and link cleanup
Related to #362
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop tp31-docs-edits
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/362
I added a separate pull request because of some differences in 3.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
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/368
TINKERPOP-1394: fixed dev list and jira links
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop TINKERPOP-1394
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/383
TINKERPOP-1246 Added help message to console
Type ':help' or ':h' for help.
You can merge this pull request into a Git repository by running:
$ git pull https://g
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/384
TINKERPOP-1037 Made life more colorful
Stephen did all the hard work.
- Added option -C to disable ANSI colors just in case.
- Colorized the gremlin ascii art.
- Added bold red to
Github user robertdale closed the pull request at:
https://github.com/apache/tinkerpop/pull/383
---
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 feature
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
Like I said, you did all the hard work. I just added color. Try this in
your console:
gremlin> "@|bold,green Hello |@ "
==>Hello
---
If your project
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
Screenshot:
---
If your project is set up for it, you
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/385
TINKERPOP-1285 added multi-line line number support
Looks like:
```
gremlin>
gremlin> 1 +
001 2 +
002 3 +
003 x
No such property: x for
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
There are some things that are out of our control because they are
inherited from groovy.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
Would you really want a separate preferences (properties) file or rather
reuse the existing preferences mechanism (:set gremlin.console.errorMessage
red) ?
---
If your project is set up for
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
## Color codes:
-BLACK
- RED
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
I think a mockup of the desired coloring effect would be nice to have.
Currently elements just have toString called on them. I added a vertex.color
and edge.color for demonstration purposes
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
I'm definitely not the right person to pick a color scheme. Open for
suggestions or we can figure it out later. Committed the vertex, edge, path
coloring so you can see.
---
If your pr
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/385
This is failing due to some random error, not the code change.
---
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
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
Attached groovy script that prints the color palette. usage: groovy
jansi-colors.txt
[jansi-colors.txt](https://github.com/apache/tinkerpop/files/427016/jansi-colors.txt
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
Disco disco! ANSI is enabled by default. This is actually not a change. It
was this way before this PR. No color scheme by default. If you want colors,
you have to set them yourself. Added
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
I think I'll leave templating for another jira ticket. :satisfied:
Currently, themes (really any `:set` commands) can be saved into files and
loaded with `:load my-gremlin.theme`.
-
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
I'm done with changes. Carry on.
---
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 fe
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/384#discussion_r75560386
--- Diff:
gremlin-console/src/main/groovy/org/apache/tinkerpop/gremlin/console/GremlinGroovysh.groovy
---
@@ -45,12 +46,13 @@ class GremlinGroovysh
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
The previous commit to fix :set with quoted spaces did break :remote
commands because the parser stripped the quotes. Since :set command is the only
one affected, I made a quick hack for it
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/384
@pluradj I noticed that too. I was thinking that metaclass could be used to
intercept calls to groovy shell's Preferences class and gremlin console could
have its own Java Preferences
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/385
Once the color preference support is added in, I can make this a
preference. The line number can be '%n' so that it can be set with `:set
multiline.prompt "%n>"`.
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/385
I would only be against `===>` because that looks like the result prompt.
I would probably lean towards `001>`. Maybe:
```
gremlin> 1 +
001> 1 +
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/385
OK. And when this is merged with the color preferences PR, it will be
adjusted and left-filled with `.` for the input.prompt length, but no less than
`000>`.
---
If your project is set
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/385
@spmallette I can rebase on latest master. Just let me know. Do you think
there is any point in keeping the leading zeroes?
```
gremlin> 1+
..1> 2+
..2> 3+
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/385
This was rebased on master.
---
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
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/391
Fixed empty result
Forgot to test this one. Fixed now.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop emptyResult
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/394
Server missing scripts directory
Fixed distribution target directory
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/395
TP31: Server missing scripts directory
Fixed distribution scripts target directory.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/395
Hold on.
---
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
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/395
OK.
---
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 user robertdale closed the pull request at:
https://github.com/apache/tinkerpop/pull/394
---
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 feature
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/399
TINKERPOP-1369 Renamed REST API to HTTP API
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop TINKERPOP-1369
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/400
I would say yes and probably through building the project. I had to add
the repo to my own project pom. That updates my local .m2 repo. Once that has
it, grapes can pull from there.
---
If
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/400
Do you also clean your groovy/grapes dir? artifacts will be copied there
from maven. afterwards, it won't reference the maven artifacts unless they are
also missing from grapes.
---
If
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/401
fixed option step header
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop ref-option-fix
Alternatively you can
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/400
Ok. Didn't notice it mentioned docker and I can't speak to that. However,
it is very repeatable on other systems. Given no other configuration...
```
rm -rf ~/.groovy
rm
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/402
fixed hardcoded link
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop fix-hardcoded-link
Alternatively you can
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/402
@spmallette done.
---
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
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/414
Release name generator
$ groovy release-name-generator.groovy
Twenty-Four Gremlin From Tulsa
$ groovy release-name-generator.groovy
(I'd) Wait a Million Gremlin
$ g
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/414
That's ok. I don't actually expect this to get merged. Just for fun. This
was a convenient way to share.
---
If your project is set up for it, you can reply to this email and have
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/423#discussion_r79032415
--- Diff: CHANGELOG.asciidoc ---
@@ -26,7 +26,9 @@
image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
TinkerPop
Github user robertdale closed the pull request at:
https://github.com/apache/tinkerpop/pull/414
---
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 feature
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/414
Inaugurated my blog and forever immortalized at
https://robertdale.github.io/2016/09/15/gremlin-create-gremlin-release-name.html
---
If your project is set up for it, you can reply to this
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/413
I'm just going to throw my $0.02 out there. Maybe I don't understand the
use case, but this seems strange and unexpected to me. I don't understand why
or how a completely
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/430
Fix store step documentation
Query produces 2 results since 3.0.0
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/433
Tested a clean console build with 3.2.2 server. Pinging works as expected.
Closing each Client, a Cluster, and ':remote close' does stop pinging as
expected.
Not sur
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/433
Use your favorite packet sniffer. I used tcpdump.
---
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
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/433
Yup, there it is right there in the javadoc: When close() is called, the
Cluster is left open for the caller to close.
---
If your project is set up for it, you can reply to this email and
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/438
Fix conf dir references
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/robertdale/tinkerpop fix-conf-references
Alternatively you can
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/439
TINKERPOP-980: added sysv and systemd init scripts
I don't know if you want all of this in bin/. I could definitely see the
systemd gremlin-server.service moving out since it's
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/438
Rebased on tp31
---
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
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/438
Build is good with `mvn clean install` on my machine.
---
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
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/439
Upstart looks simple enough. I don't know if the one @dkuppitz created,
referenced in TINKERPOP-980 , is still viable. But I can't test. Any volunteers?
I would have to say
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/439
Maybe if the script detects that it's called via link with no parameters,
then it's in initd mode and displays help. If the script is called directly
with no parameters, then it ju
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/444#discussion_r81139237
--- Diff: gremlin-console/src/main/bin/gremlin.sh ---
@@ -111,3 +115,5 @@ fi
# Start the JVM, execute the application, and return its exit
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/439
- `gremlin-server.sh` _is_ the init script. This makes it more inline with
other Apache servers like Tomcat, ActiveMQ, solr, etc.
- Moved how to install "As A Service" from
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/439
Cool. I was wondering how to link. Didn't think it was absolute. I hope it
works. Tried testing with `./docker/build.sh -d` but it hangs at installing
plugins `* hadoop-gr
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/439
That should still work. What won't work is an invocation of just
"gremlin-server.sh" without parameters. I didn't find any in the code base nor
in the docs (which
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/439
I just tested with backgrounding the job and killing that. That also still
works.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/452
TINKERPOP-1189 Increased performance of CredentialGraph authentication
Using lowest complexity 4 (was 10).
You can merge this pull request into a Git repository by running:
$ git pull
301 - 390 of 390 matches
Mail list logo
