Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/742
@spmallette Updated NOTICE with latest version. No new deps.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/745
The race condition is in `put()` and it's used elsewhere. It would make
more sense to fix `put()` to be thread-safe. Then we can keep
`parallelStream()`
```
Map<Obj
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/742
TINKERPOP-1824 Bump netty to 4.0.52
https://issues.apache.org/jira/browse/TINKERPOP-1824
Latest bug-fix release.
VOTE +1
You can merge this pull request into a Git
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/744
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/763
Yeah, I didn't initially see it in tp32. This is not the right approach
anyway. Will push an update.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/763
`docker/build.sh -t` BUILD SUCCESS
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/765
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/763
Fixed test only.
Tests pass.
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/736
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/737
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/715
@mpollmeier Making it official. Please update the title to start with
`TINKERPOP-1822`
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/739
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/862
After running `bin/process-docs.sh` the resulting console/server zip files
no longer contain `docs/` let alone `images`. Is there a different step that
needs to be run?
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/862
I did:
1. `bin/process-docs.sh`
2. `mvn install`
3. `find . -name '*.zip'`
4. `unzip -t
./gremlin-server/target/apache-tinkerpop-gremlin-server-3.2.9-distribution.zip
| grep
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/862
I walked through the actual release process and this time I do see the
correct packaging.
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/869
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/860
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/864
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/861
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/874
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/871
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
@dkuppitz I did see that. There was a dep added to the image. I have that
rebuilding now. Will let you know the results.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
@dkuppitz I pushed a CTR fix for invisibility to tp32/master. This fixes
the screen from being rendered to pixels wide. It didn't help the code
block language popup though.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
Try rebuilding the docker images using master.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
Hmm.. there doesn't seem to be much difference. Are you deleting both the
hadoop and base images?
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/767
Failed docker...
```
docker/build.sh -i -t
[ERROR] Tests run: 5, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
13.448 s <<<
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/767
Is it just the wrong expected exception in the test?
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
@spmallette did you save your console log?
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
In the 'Console' tab, muli-line statements work. They have the `\`
continuation after the callout. However, on the 'language' tab, these seem to
be lost thus copy-paste multi-line statements
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/767
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/773
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/771
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/778
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/781
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/789
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/791
Those tests pass? It doesn't seem to work for me on the cli.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/795
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/794
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/793
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/799
VOTE +1
---
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/800#discussion_r169304285
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/AbstractChannelizer.java
---
@@ -147,7 +149,12 @@ public void init(final
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/800#discussion_r169305818
--- Diff:
gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerIntegrateTest.java
---
@@ -293,6 +312,21 @@ private static
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/800#discussion_r169302920
--- Diff: docs/src/reference/gremlin-applications.asciidoc ---
@@ -1073,7 +1073,8 @@ The following table describes the various
configuration options
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/800#discussion_r169302646
--- Diff: CHANGELOG.asciidoc ---
@@ -25,6 +25,7 @@
image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/800#discussion_r169364908
--- Diff: docs/src/reference/gremlin-applications.asciidoc ---
@@ -1073,7 +1073,8 @@ The following table describes the various
configuration options
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/800
Yes, manually. I was surprised too. In practice, I don't think it's a big
deal. The only reason I could see even using server-side keep-alive is because
client-side keep-alive was sufficient
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/800
The only thing I don't like is that the client keep-alive doesn't seem to
reset the 'write idle time' so the server still sends ping requests to the
client even though the client
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
What about for those of us unable to build it?
```
sh docker/build.sh -d
* source: /usr/src/tinkermem/docs/src/recipes/olap-spark-yarn.asciidoc
target
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
Just what I was afraid of... the tabs look just like they do in the
screenshots. And I don't mean that in a good way :smile:
Seriously, this is fantastic. Multi-line works too
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/766
Excellent!
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/768
I don't think we should break the interface. I think these should be new
methods and `@deprecate` the old ones. The old methods can call the new methods.
---
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/904
TINKERPOP-2016 Bumped to Jackson 2.9.6
https://issues.apache.org/jira/browse/TINKERPOP-2016
Bumped to Jackson 2.9.6 for CVE.
You can merge this pull request into a Git repository
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/912
TINKERPOP-2023 SSL Enhancements
https://issues.apache.org/jira/browse/TINKERPOP-2023
Gremlin Server no longer supports automatically creating self-signed
certificates
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/907
@RainMark Could you create an issue in Jira and describe the problem? See
also [Contributing code
changes](https://tinkerpop.apache.org/docs/current/dev/developer/#_contributing_code_changes
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/912
@k4rthikr That's a little deeper than I want to get into. `sslCipherSuites`
provides a whitelist. Whitelists are always preferred over blacklists. If this
one were blacklisted
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/912
Right, when it's security-related, you're given a pass to break anything!
:rage1: :rage2: :rage3: :rage4:
Seriously, the breaking change is only if you relied on the server
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/919
@dkuppitz Will do.
---
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/919#discussion_r211952645
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/DefaultTraversalMetrics.java
---
@@ -208,14 +208,14
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/919
@otaviojava did you decompile the bytecode to make sure the compiler didn't
do that automagically? Otherwise, StringBuilder vars should be final.
---
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/919#discussion_r211948320
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/DefaultTraversalMetrics.java
---
@@ -208,14 +208,14
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/919
@otaviojava was that locally? the travis build failed due to grapes
download.
---
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/919#discussion_r211934275
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/DefaultTraversalMetrics.java
---
@@ -286,10 +286,10
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/919#discussion_r211934120
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/DefaultTraversalMetrics.java
---
@@ -208,14 +208,14
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/919
Hmm.. it happens on tp32.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/919
Nevermind, still just grapes.
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/914
VOTE +1
---
GitHub user robertdale opened a pull request:
https://github.com/apache/tinkerpop/pull/930
TINKERPOP-2032 bump jython-standalone 2.7.1
https://snyk.io/vuln/SNYK-JAVA-ORGPYTHON-31451
Overview
org.python:jython-standalone Affected versions of this package are
vulnerable
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/876
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/886
@spmallette any idea why the spark test is hanging in travis?
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/775
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/784
VOTE +1
---
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/784#discussion_r163528577
--- Diff: docs/src/dev/io/graphson.asciidoc ---
@@ -1279,7 +1270,7 @@ The following `RequestMessage` is an example of a
sessionless request for a scri
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/784#discussion_r163529709
--- Diff: docs/src/dev/io/graphson.asciidoc ---
@@ -75,8 +72,7 @@ file.withWriter { writer ->
writer.write(toJson(msg, "Sessionl
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/784#discussion_r163529404
--- Diff: docs/src/dev/io/graphson.asciidoc ---
@@ -4066,11 +4075,12 @@ The following `RequestMessage` is an example of a
sessionless request
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/784#discussion_r163529843
--- Diff: docs/src/dev/io/graphson.asciidoc ---
@@ -187,8 +180,7 @@ file.withWriter { writer ->
writer.write(toJson(msg, "Sessionl
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/786
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/812
What's the difference between the global method cache and the local method
cache?
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/808
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/803
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/809
```
$ docker/build.sh -i -t -n
[INFO]
[INFO] BUILD SUCCESS
[INFO
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/829#discussion_r179166260
--- Diff: docs/src/reference/the-traversal.asciidoc ---
@@ -1515,16 +1516,17 @@
link:++http://tinkerpop.apache.org/javadocs/x.y.z/core/org/apache
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/829#discussion_r179166002
--- Diff: docs/src/reference/the-traversal.asciidoc ---
@@ -1471,16 +1471,17 @@
link:++http://tinkerpop.apache.org/javadocs/x.y.z/core/org/apache
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/837
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/836
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/830
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/838
Look at the Serializers in
`./gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/`
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/835
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/833
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/832
VOTE +1
---
Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/838#discussion_r181408164
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatStep.java
---
@@ -273,11 +314,37 @@ public
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/829
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/839
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/855
I think `computeIfAbsent()` would have been better :goat:
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/857
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/855
Well, you would do use ConcurrentHashMap ;-)
```
static Map<Class, Boolean> LOADED = new ConcurrentHashMap<>();
try {
LOADED.com
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/851
@jorgebay don't forget to vote on your own PR! :smile:
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/853
VOTE +1
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/826
Looks like netty is upgraded in Spark 2.3.0 only.
https://issues.apache.org/jira/browse/SPARK-19552
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/826
Can this be closed now?
---
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/825
Can this be merged?
---
201 - 300 of 338 matches
Mail list logo