[GitHub] [tinkerpop] otaviojava commented on issue #951: Optimizes Set with enum using the EnumSet implementation [tp32]
you welcome [ Full content available at: https://github.com/apache/tinkerpop/pull/951 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[jira] [Closed] (TINKERPOP-2057) TypeError when grouping by multiple attributes
[
https://issues.apache.org/jira/browse/TINKERPOP-2057?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette closed TINKERPOP-2057.
---
Resolution: Not A Problem
sorry - took me a while to get to digging into this. upon investigation, i
don't think that there's a problem here with Gremlin. It's an issue with the
way dictionaries work native to Python. So here's your output:
{code}
gremlin> g.V().as("a").out().out().as("b").path().group().by(__.select("a",
"b")).unfold()
==>{a=v[1], b=v[3]}=[[v[1], v[4], v[3]]]
==>{a=v[1], b=v[5]}=[[v[1], v[4], v[5]]]
{code}
You're returning a dictionary that uses a dictionary as a key. Python
dictionary is immutable and therefore doesn't work as a key:
https://stackoverflow.com/q/20115680/1831717
It seems the dict could be "frozen" according to that answer, but I don't think
we will do that. I think that this is just a limitation of the Gremlin language
in python and that you should just alter your result structure to return a key
that works for dictionaries in python.
Since this is behavior is expected of Python, I'm going to close this as it's
not a bug - if you'd like to discuss it further, please bring up your concerns
on the dev list.
https://lists.apache.org/list.html?dev@tinkerpop.apache.org
> TypeError when grouping by multiple attributes
> --
>
> Key: TINKERPOP-2057
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2057
> Project: TinkerPop
> Issue Type: Bug
> Components: python
>Affects Versions: 3.3.3
>Reporter: Pierce Freeman
>Priority: Major
>
> The python serialization engine fails when trying to group a path by two
> intermediate variables:
>
> {code:java}
> graph = Graph()
> g =
> graph.traversal().withRemote(DriverRemoteConnection('ws://localhost:8182/gremlin','g'))
> g.V().as_("a").out().out().as_("b").path().group().by(__.select("a",
> "b")).unfold().toList()
> {code}
> {code:java}
> ~/tinkerpop/gremlin-python/src/main/jython/gremlin_python/structure/io/graphsonV3d0.py
> in objectify(cls, l, reader)
> 456 while x < len(l):
> 457 new_dict[reader.toObject(l[x])] = reader.toObject(l[x + 1])
> --> 458 x = x + 2
> 459 return new_dict
> 460
> TypeError: unhashable type: 'dict'
> {code}
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[jira] [Assigned] (TINKERPOP-2059) Modulation of valueMap()
[
https://issues.apache.org/jira/browse/TINKERPOP-2059?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Daniel Kuppitz reassigned TINKERPOP-2059:
-
Assignee: Daniel Kuppitz
> Modulation of valueMap()
>
>
> Key: TINKERPOP-2059
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2059
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
>Affects Versions: 3.3.3
>Reporter: stephen mallette
>Assignee: Daniel Kuppitz
>Priority: Major
>
> Allow for modulation of the {{valueMap()}} step with {{by()}} and possibly
> {{with()}} to make it's usage more flexible and consistent. The problem
> presented in relation to this change is how to easily unwrap the general
> multi-property structure of {{valueMap()}}
> {code}
> g.V().has('person','name','marko').
> valueMap('name','age').
> by(unfold())
> {code}
> which would yield: {{[name:marko,age:29]}} rather than:
> {{[name:[marko],age:[29]]}}. Obviously the {{by()}} is just a map over the
> values in the list of values returned from properties.
> Please see the
> [DISCUSS|https://lists.apache.org/thread.html/91a3c980b5968c78638383dfa11f44ee14fc114ddc3d29e762e56f1f@%3Cdev.tinkerpop.apache.org%3E]
> thread for additional thoughts on the matter to consider.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[jira] [Commented] (TINKERPOP-2061) Add with() configuration as global to a traversal
[
https://issues.apache.org/jira/browse/TINKERPOP-2061?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16659581#comment-16659581
]
ASF GitHub Bot commented on TINKERPOP-2061:
---
spmallette closed pull request #961: TINKERPOP-2061 Added with() option for
traversal configuration
URL: https://github.com/apache/tinkerpop/pull/961
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Add with() configuration as global to a traversal
> -
>
> Key: TINKERPOP-2061
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2061
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
>Affects Versions: 3.4.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Major
> Fix For: 3.4.0
>
>
> Provide a for {{with(String)}} and {{with(String,Object)}} at the
> configuration of the {{TraversalSource}} - basically a shorthand for using
> {{OptionsStrategy}} manually and {{withStrategies()}}.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[jira] [Commented] (TINKERPOP-2061) Add with() configuration as global to a traversal
[
https://issues.apache.org/jira/browse/TINKERPOP-2061?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16659563#comment-16659563
]
ASF GitHub Bot commented on TINKERPOP-2061:
---
spmallette commented on a change in pull request #961: TINKERPOP-2061 Added
with() option for traversal configuration
URL: https://github.com/apache/tinkerpop/pull/961#discussion_r227050936
##
File path:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/TraversalSource.java
##
@@ -93,6 +96,39 @@ private Symbols() {
/
+/**
+ * Provides a configuration to a traversal in the form of a key which is
the same as {@code with(key, true)}. The
+ * key of the configuration must be graph provider specific and therefore
a configuration could be supplied that
+ * is not known to be valid until execution.
+ *
+ * @param key the key of the configuration to apply to a traversal
+ * @return a new traversal source with the included configuration
+ * @since 3.4.0
+ */
+public default TraversalSource with(final String key) {
+return with(key, true);
+}
+
+/**
+ * Provides a configuration to a traversal in the form of a key value
pair. The key of the configuration must be
+ * graph provider specific and therefore a configuration could be supplied
that is not known to be valid until
+ * execution. This is a handy shortcut for building an {@link
OptionsStrategy} manually and then add with
+ * {@link #withStrategies(TraversalStrategy[])}.
+ *
+ * @param key the key of the configuration to apply to a traversal
+ * @param value the value of the configuration to apply to a traversal
+ * @return a new traversal source with the included configuration
+ * @since 3.4.0
+ */
+public default TraversalSource with(final String key, final Object value) {
+final Optional opt =
getStrategies().getStrategy(OptionsStrategy.class);
+final OptionsStrategy strategy = opt.orElse(OptionsStrategy.EMPTY);
+final OptionsStrategy.Builder builder = OptionsStrategy.build();
+strategy.getOptions().forEach(builder::with);
+builder.with(key, value);
+return withStrategies(builder.create());
+}
Review comment:
done
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Add with() configuration as global to a traversal
> -
>
> Key: TINKERPOP-2061
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2061
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
>Affects Versions: 3.4.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Major
> Fix For: 3.4.0
>
>
> Provide a for {{with(String)}} and {{with(String,Object)}} at the
> configuration of the {{TraversalSource}} - basically a shorthand for using
> {{OptionsStrategy}} manually and {{withStrategies()}}.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[GitHub] [tinkerpop] spmallette commented on pull request #961: TINKERPOP-2061 Added with() option for traversal configuration
done [ Full content available at: https://github.com/apache/tinkerpop/pull/961 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] spmallette commented on issue #951: Optimizes Set with enum using the EnumSet implementation [tp32]
I've merged this to tp33/master. We've closed off tp32 to all but major bug fixes at this point. Thanks for doing this work @otaviojava [ Full content available at: https://github.com/apache/tinkerpop/pull/951 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] spmallette closed pull request #951: Optimizes Set with enum using the EnumSet implementation [tp32]
[ pull request closed by spmallette ] [ Full content available at: https://github.com/apache/tinkerpop/pull/951 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[jira] [Commented] (TINKERPOP-2071) gremlin-python: the graphson deserializer for g:Set should return a python set
[ https://issues.apache.org/jira/browse/TINKERPOP-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16659531#comment-16659531 ] ASF GitHub Bot commented on TINKERPOP-2071: --- spmallette commented on issue #964: TINKERPOP-2071: gremlin-python: g:Set graphson deserializer should return a python set URL: https://github.com/apache/tinkerpop/pull/964#issuecomment-431882499 Thanks for submitting thisTravis seems unhappy on the gremlin-python version of the buildcould you please have a look at that? Also, this looks like it needs a rebase now that we have the tp33 branch re-opened for development. That should get CHANGELOG setup for new entries. Please don't include the JIRA issue there - just a bullet point with your representation of what changed will suffice. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > gremlin-python: the graphson deserializer for g:Set should return a python set > -- > > Key: TINKERPOP-2071 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2071 > Project: TinkerPop > Issue Type: Improvement > Components: python >Affects Versions: 3.3.3 >Reporter: Alan Boudreault >Assignee: stephen mallette >Priority: Major > > This is something that has already been discussed. The reason that a g:Set is > deserialized to a list is because Java can return some numerical values that > are not differentiate by python. See TINKERPOP-1844 for more details. > However, I think this is is not something very common so I would like to > propose another behavior: > 1. By default, always return a python set as normal for g:Set > 2. When we detect the case described, log a warning and return a python list. > 3. Document properly the limitation and the behavior of g:Set. > Already discussed with Stephen and we think it's an acceptable behavior. I > will provide a PR shortly. > -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] [tinkerpop] spmallette commented on issue #964: TINKERPOP-2071: gremlin-python: g:Set graphson deserializer should return a python set
Thanks for submitting thisTravis seems unhappy on the gremlin-python version of the buildcould you please have a look at that? Also, this looks like it needs a rebase now that we have the tp33 branch re-opened for development. That should get CHANGELOG setup for new entries. Please don't include the JIRA issue there - just a bullet point with your representation of what changed will suffice. [ Full content available at: https://github.com/apache/tinkerpop/pull/964 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] spmallette commented on issue #963: tiny typo, change plural to singular in MapReduce explanation
cherry-picked back to tp33 - 6cadd076877237e35de9b24388ea1b063dc37f56 thanks! [ Full content available at: https://github.com/apache/tinkerpop/pull/963 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] spmallette closed pull request #963: tiny typo, change plural to singular in MapReduce explanation
[ pull request closed by spmallette ] [ Full content available at: https://github.com/apache/tinkerpop/pull/963 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] spmallette commented on issue #965: TINKERPOP-2073 Generate tabs for static code blocks
should this be targetted on tp33 - we have tabs there too, right? other than that VOTE +1 [ Full content available at: https://github.com/apache/tinkerpop/pull/965 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[jira] [Commented] (TINKERPOP-2073) Generate tabs for static code blocks
[ https://issues.apache.org/jira/browse/TINKERPOP-2073?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16659514#comment-16659514 ] ASF GitHub Bot commented on TINKERPOP-2073: --- spmallette commented on issue #965: TINKERPOP-2073 Generate tabs for static code blocks URL: https://github.com/apache/tinkerpop/pull/965#issuecomment-431875804 should this be targetted on tp33 - we have tabs there too, right? other than that VOTE +1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Generate tabs for static code blocks > > > Key: TINKERPOP-2073 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2073 > Project: TinkerPop > Issue Type: Improvement > Components: documentation >Affects Versions: 3.4.0 >Reporter: Daniel Kuppitz >Assignee: Daniel Kuppitz >Priority: Major > > Tabs are only generated for evaluated code blocks. It should be possible to > tabify static code blocks, too. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
Re: [DISCUSS] ReferenceStrategy
https://issues.apache.org/jira/browse/TINKERPOP-2075 On Mon, Oct 8, 2018 at 1:37 PM Stephen Mallette wrote: > > I'd like to propose 2a. Update Gremlin Server and TinkerGraph to behave > in the desired way, this would set the tone for all Graph implementations > to eventually adopt this consistent behaviour. > > a little scary, but i see your point. if we went that far then i guess we > would also do neo4j. i will say that this change will be on the same order > of effort as 1 because it will break a lot of tests in the test suite. in > that sense, i'm not sure we're ready for that much change along 3.4.0. i'd > propose that we consider "2a" only after we get "2" into place. Then we can > determine how much damage 1 would do. Maybe it's not as bad as I think. An > interesting side-effect of 1 is that it makes our Java test suite tests > become more in line with the GLV tests - they would be roughly 1 to 1 match > in assertions after all this. > > > On Fri, Oct 5, 2018 at 11:21 AM brynco...@gmail.com > wrote: > >> >> >> On 2018/10/03 17:06:16, Stephen Mallette wrote: >> > We currently have this situation where users get a fair bit of >> > inconsistency around the contents of graph elements depending on a >> matrix >> > of different usage options that we offer - here's just a few "options" >> as >> > examples: >> > >> > 1. Use embedded graph mode in OLTP and you likely get the implementation >> > version of a Vertex/Edge with accessible properties (e.g. TinkerVertex, >> > Neo4jVertex, etc) >> > 2. Use embedded graph mode in OLAP and you get ReferenceVertex/Edge >> with no >> > properties >> > 3. Use bytecode based requests with Gremlin Server and you get >> > ReferenceVertex/Edge with no properties >> > 4. Use script based requests with Gremlin Server and you get >> > DetachedVertex/Edge with properties >> > >> > All this chaos developed out of a healthy evolution of our view of "how >> > Gremlin should work and how it fits in the graph community." As I've >> > lamented before and I'll lament again, that if we'd foreseen "bytecode", >> > then a lot of things would have been more unified. >> > >> > Anyway, irrespective of how we got here, I think this matrix of choices >> > ends up making things quite confusing for users. >> > >> > To unify and simplify in 3.4.0, I think we could introduce a >> > ReferenceStrategy which would coerce graph elements to the "Reference". >> > Then we have some choices: >> > >> > 1. In the most extreme case, it could be installed as a default >> strategy. >> > All graphs was return the same stuff in whatever situation it was used. >> > Obviously, that breaks just about every test in existence and probably >> half >> > of the code on the planet that uses TinkerPop - everyone ready to not >> get >> > amazon packages delivered on time anymore over this? But, we're >> consistent! >> > :) >> > 2. We install it as a default strategy in graphs in Gremlin Server. >> Still a >> > breaking change but at least Gremlin Server is completely consistent. >> Users >> > can uninstall the strategy if they don't like it and stuff will work as >> it >> > always did. >> > 3. We simply supply ReferenceStrategy as an option and let users >> install it >> > for themselves to help bring greater consistency to their installations. >> > >> > I think we should consider option 2. Unless someone has other options to >> > consider, it seems like the easiest starting point for this that will >> > actually have an impact. There could be devils lying in wait in the >> > details, but I thought I'd feel folks on a bit on the general idea. >> > >> > Thanks, >> > >> > Stephen >> > >> >> >> The thing that worries me is that to go for option 2 still leaves >> TinkerGraph behaving in a way that is not consistent over the wire. >> >> The first thing that many users will do is fire up gremlin console and >> create a TinkerGraph without Gremlin Server. This immediately give them a >> false impression that properties on elements are available. >> >> I'd like to propose 2a. Update Gremlin Server and TinkerGraph to behave >> in the desired way, this would set the tone for all Graph implementations >> to eventually adopt this consistent behaviour. >> >> Bryn >> >
[jira] [Created] (TINKERPOP-2075) Introduce DetachmentStrategy
stephen mallette created TINKERPOP-2075:
---
Summary: Introduce DetachmentStrategy
Key: TINKERPOP-2075
URL: https://issues.apache.org/jira/browse/TINKERPOP-2075
Project: TinkerPop
Issue Type: Improvement
Components: process
Affects Versions: 3.4.0
Reporter: stephen mallette
Introduce {{DetachmentStrategy}} to auto detach graph elements. We would
probably have option to configure the strategy to use "detached" or "reference"
elements with the default being "reference" (i.e. no properties). This will
start to help to align all usage of Gremlin around "reference" elements no
matter how users connect (i.e. GLVs, embedded, Gremlin Server, etc.).
Initially, this issue should just be about implementing the strategy and
getting it into play with the rest of the things it relates to. Not sure what
it does to {{HaltedTraverserStrategy}} if anything at this point or the various
detachment methods that already exist.
Once the feature is in place we can decide how it will be implemented by
default in the framework - see the linked DISCUSS thread for more information.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[jira] [Closed] (TINKERPOP-2061) Add with() configuration as global to a traversal
[
https://issues.apache.org/jira/browse/TINKERPOP-2061?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette closed TINKERPOP-2061.
---
Resolution: Done
Fix Version/s: 3.4.0
> Add with() configuration as global to a traversal
> -
>
> Key: TINKERPOP-2061
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2061
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
>Affects Versions: 3.4.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Major
> Fix For: 3.4.0
>
>
> Provide a for {{with(String)}} and {{with(String,Object)}} at the
> configuration of the {{TraversalSource}} - basically a shorthand for using
> {{OptionsStrategy}} manually and {{withStrategies()}}.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
Re: code freeze 3.2.10/3.3.4
there's always going to be those kinds of things though right? can we get
away with doing stuff to tp32 only when there is some specific demand for a
3.2.11. like, that fix on its own doesn't feel like something we'd trigger
a release overor would we?
On Sat, Oct 20, 2018 at 9:27 AM Robert Dale wrote:
> You don't want to put https://issues.apache.org/jira/browse/TINKERPOP-2068
> on tp32?
>
> Robert Dale
>
>
> On Fri, Oct 19, 2018 at 6:03 PM Stephen Mallette
> wrote:
>
> > > I created a ticket to track this and I can probably take care of it
> next
> > week
> >
> > that makes sense, thank you.
> >
> > separately, code freeze is now lifted on tp33 - i've bumped to
> > 3.3.5-SNAPSHOT, published initial docs/artifacts and all is good to go.
> > I've left tp32 on 3.2.10 until we decide to actually do something on that
> > branch. For now, we'll just say we're done there as we discussed
> > elsethread.
> >
> > kuppitz, feel free to fire up the dead branch cleanup email. i wonder if
> it
> > will be more convenient to delete branches as we go now that we have the
> > github UI available to us. might be good to just hit the "delete branch"
> > button right when we hit the merge one.
> >
> > i will announce the releases on monday morning EST time.
> >
> >
> >
> > On Fri, Oct 19, 2018 at 5:04 PM Florian Hockmann >
> > wrote:
> >
> > > We currently use this to push the package:
> > >
> > > ${nugetExe} push Gremlin.Net/bin/Gremlin.Net.*.nupkg
> > >
> > > which pushes all NuGet packages starting with Gremlin.Net. You probably
> > > also had the package for version 3.4.0-rc2 in that directory. So,
> > > nuget.exe tried to push that version again which isn't allowed by NuGet
> > > as packages are immutable for a given version.
> > >
> > > I guess we should specify the exact version here instead to avoid these
> > > problems in the future. Otherwise we could push development versions to
> > > nuget.org by accident. I probably implemented it like this at first
> > > because I assumed that mvn clean would always remove older packages
> > > which seems to be not the case here.
> > >
> > > Anyway, I created a ticket to track this and I can probably take care
> of
> > > it next week:
> > >
> > > https://issues.apache.org/jira/browse/TINKERPOP-2074
> > >
> > >
> > > Am 19.10.2018 um 21:16 schrieb Stephen Mallette:
> > > > Florian, any idea what's going on with this error when i deployed
> > 3.3.4:
> > > >
> > > > main:
> > > > [echo] nuget.exe already downloaded.
> > > > [exec] Pushing Gremlin.Net.3.3.4.nupkg to '
> > > > https://www.nuget.org/api/v2/package'...
> > > > [exec] PUT https://www.nuget.org/api/v2/package/
> > > > [exec] Created https://www.nuget.org/api/v2/package/ 1362ms
> > > > [exec] Your package was pushed.
> > > > [exec] Pushing Gremlin.Net.Template.3.3.4.nupkg to '
> > > > https://www.nuget.org/api/v2/package'...
> > > > [exec] PUT https://www.nuget.org/api/v2/package/
> > > > [exec] Created https://www.nuget.org/api/v2/package/ 11405ms
> > > > [exec] Your package was pushed.
> > > > [exec] Pushing Gremlin.Net.Template.3.4.0-rc2.nupkg to '
> > > > https://www.nuget.org/api/v2/package'...
> > > > [exec] PUT https://www.nuget.org/api/v2/package/
> > > > [exec] Conflict https://www.nuget.org/api/v2/package/ 365ms
> > > > [exec] 409 (A package with ID 'Gremlin.Net.Template' and version
> > > > '3.4.0-rc2' already exists and cannot be modified.)
> > > >
> > > >
> > > > why is it trying to push the 3.4.0 line? looks like the error didn't
> > > > matter as the other two published fine, but let's fix the error if we
> > > > can...
> > > >
> > > > On Fri, Oct 12, 2018 at 7:27 PM Stephen Mallette <
> spmalle...@gmail.com
> > >
> > > > wrote:
> > > >
> > > >> Robert Dale is without power so I pushed the 3.2.10-SNAPSHOT
> artifacts
> > > and
> > > >> docs too:
> > > >>
> > > >> http://tinkerpop.apache.org/docs/3.2.10-SNAPSHOT/
> > > >>
> > > >>
> > > >>
> > > >> On Fri, Oct 12, 2018 at 8:22 AM Stephen Mallette <
> > spmalle...@gmail.com>
> > > >> wrote:
> > > >>
> > > >>> I just published final 3.3.4-SNAPSHOT artifacts and docs:
> > > >>>
> > > >>> http://tinkerpop.apache.org/docs/3.3.4-SNAPSHOT/
> > > >>>
> > > >>> On Sun, Oct 7, 2018 at 8:33 AM Robert Dale
> > wrote:
> > > >>>
> > > I'll take 3.2.10.
> > >
> > > Robert Dale
> > >
> > >
> > > On Fri, Oct 5, 2018 at 7:40 PM Stephen Mallette <
> > spmalle...@gmail.com
> > > >
> > > wrote:
> > >
> > > > So - code freeze for the tp32 and tp33 branches is in effect at
> > this
> > > point.
> > > > We'll use this thread to discuss issues related to 3.2.10 and
> 3.3.4
> > > leading
> > > > up to the build of release artifacts for 10/15. We currently
> have a
> > > few PRs
> > > > that need merging to those branches,
> > > >
> > > > https://github.com/apache/tinkerpop/pull/952 (submit scripts
> with
> > >
Re: Development branch cleanup
yeah - hopefully we can just remember to click the "delete branch" after we merge. that's pretty easy, but i guess it's worth running this little script anyway on release just in case something gets missed. hopefully the list will be smaller in the future to make it easier to reason about. On Mon, Oct 22, 2018 at 11:29 AM Jorge Bay Gondra wrote: > lgtm > > I think that with GitBox enabled, going forward, we should be able to > delete our branches, right? > > El lun., 22 oct. 2018 a las 16:56, Daniel Kuppitz () > escribió: > > > The following branches are scheduled for deletion: > > > > TINKERPOP-1342 -- [TINKERPOP-1342] Allow setting scriptEvaluationTimeout > in > > driver > > TINKERPOP-1365 -- [TINKERPOP-1365] Log the seed used to initialize Random > > in tests > > TINKERPOP-1489 -- [TINKERPOP-1489] Provide a Javascript Gremlin Language > > Variant > > TINKERPOP-1518 -- [TINKERPOP-1518] Provide a way for providers to expose > > static Graph.Features to tests > > TINKERPOP-1595 -- [TINKERPOP-1595] Go through TraversalVertexProgram > with a > > profile and optimize. > > TINKERPOP-1656 -- [TINKERPOP-1656] Write-time compilation of mutation > steps > > TINKERPOP-1730 -- [TINKERPOP-1730] Gremlin .NET support for GraphSON 3.0 > > TINKERPOP-1738-wip -- [TINKERPOP-1738] Proper functioning of > GraphSONReader > > depends on order of elements in String representation > > TINKERPOP-1744 -- [TINKERPOP-1744] Gremlin .NET: Exception from sync > > execution gets wrapped in AggregateException > > TINKERPOP-1809 -- [TINKERPOP-1809] Process tests shouldn't modify toy > data > > TINKERPOP-1820 -- [TINKERPOP-1820] Include .NET GLV tests on TravisCI > > TINKERPOP-1827 -- [TINKERPOP-1827] Gremlin .NET: Test Suite Runner > > TINKERPOP-1831 -- [TINKERPOP-1831] Refactor EventStrategy > > TINKERPOP-1836 -- [TINKERPOP-1836] .NET sample project > > TINKERPOP-1837 -- [TINKERPOP-1837] Gremlin .NET: Provide type coercion > > between IDictionary instances > > TINKERPOP-1841 -- [TINKERPOP-1841] Include Python GLV tests on TravisCI > > TINKERPOP-1864 -- [TINKERPOP-1864] Gremlin Python tests for GraphSON 2.0 > > and 3.0 > > TINKERPOP-1878 -- [TINKERPOP-1878] Sparql to Gremlin Compiler > > TINKERPOP-1880 -- [TINKERPOP-1880] Gremlin.NET Strong name signature > could > > not be verified. (HRESULT: 0x80131045) > > TINKERPOP-1897 -- [TINKERPOP-1897] Provide Docker images of Gremlin > Server > > and Console > > TINKERPOP-1913 -- [TINKERPOP-1913] Expose metadata from Gremlin Server to > > Clients > > TINKERPOP-1945 -- [TINKERPOP-1945] Add support for extended GraphSon > types > > to Gremlin.net > > TINKERPOP-1958 -- [TINKERPOP-1958] TinkerGraphCountStrategy can return > > wrong counts > > TINKERPOP-1959 -- [TINKERPOP-1959] Provide a way to submit scripts to the > > server in gremlin-javascript > > TINKERPOP-1959-master -- [TINKERPOP-1959] Provide a way to submit scripts > > to the server in gremlin-javascript > > TINKERPOP-1959-tp33 -- [TINKERPOP-1959] Provide a way to submit scripts > to > > the server in gremlin-javascript > > TINKERPOP-1963 -- [TINKERPOP-1963] Use of reducing step in choose() > > TINKERPOP-1967 -- [TINKERPOP-1967] Add a connectedComponent() step > > TINKERPOP-1968 -- [TINKERPOP-1968] Refactor elements of Gremlin Server > > testing > > TINKERPOP-1972 -- [TINKERPOP-1972] inject() tests are throwing exceptions > > in .NET GLV tests > > TINKERPOP-1972-master -- [TINKERPOP-1972] inject() tests are throwing > > exceptions in .NET GLV tests > > TINKERPOP-1972-tp33 -- [TINKERPOP-1972] inject() tests are throwing > > exceptions in .NET GLV tests > > TINKERPOP-1975 -- [TINKERPOP-1975] Introduce with() step modulator > > TINKERPOP-1975-x -- [TINKERPOP-1975] Introduce with() step modulator > > TINKERPOP-1976 -- [TINKERPOP-1976] Include Computer tests for GLVs > > TINKERPOP-1978 -- [TINKERPOP-1978] Check for Websocket connection state > > when retrieved from Connection Pool missing > > TINKERPOP-1979 -- [TINKERPOP-1979] Several OLAP issues in MathStep > > TINKERPOP-1985 -- [TINKERPOP-1985] Update position on bulk loading > > TINKERPOP-1986 -- [TINKERPOP-1986] Remove deprecation from > > PartitionStrategy, SubgraphStrategy and GremlinScriptEngine > > TINKERPOP-1987 -- [TINKERPOP-1987] Bump to Netty 4.1.x > > TINKERPOP-1989 -- [TINKERPOP-1989] Preserve order that plugins are > applied > > in Gremlin Console > > TINKERPOP-1990 -- [TINKERPOP-1990] Add a shortestPath() step > > TINKERPOP-1996 -- [TINKERPOP-1996] Introduce read() and write() steps > > TINKERPOP-1999 -- [TINKERPOP-1999] [Java][gremlin-driver] Query to a > remote > > server via the websocket client hangs indefinitely if the server becomes > > unavailable > > TINKERPOP-2011 -- [TINKERPOP-2011] Use NumberHelper on choose() > > TINKERPOP-2012 -- [TINKERPOP-2012] Target .NET Standard 2.0 for > Gremlin.Net > > TINKERPOP-2015 -- [TINKERPOP-2015] Allow users to configure the WebSocket > > connections > > TINKERPOP-2016 -- [TINKERPOP-2016] Upgrade Jackson FasterXML to 2.9.5 or > > late
Re: Development branch cleanup
lgtm I think that with GitBox enabled, going forward, we should be able to delete our branches, right? El lun., 22 oct. 2018 a las 16:56, Daniel Kuppitz () escribió: > The following branches are scheduled for deletion: > > TINKERPOP-1342 -- [TINKERPOP-1342] Allow setting scriptEvaluationTimeout in > driver > TINKERPOP-1365 -- [TINKERPOP-1365] Log the seed used to initialize Random > in tests > TINKERPOP-1489 -- [TINKERPOP-1489] Provide a Javascript Gremlin Language > Variant > TINKERPOP-1518 -- [TINKERPOP-1518] Provide a way for providers to expose > static Graph.Features to tests > TINKERPOP-1595 -- [TINKERPOP-1595] Go through TraversalVertexProgram with a > profile and optimize. > TINKERPOP-1656 -- [TINKERPOP-1656] Write-time compilation of mutation steps > TINKERPOP-1730 -- [TINKERPOP-1730] Gremlin .NET support for GraphSON 3.0 > TINKERPOP-1738-wip -- [TINKERPOP-1738] Proper functioning of GraphSONReader > depends on order of elements in String representation > TINKERPOP-1744 -- [TINKERPOP-1744] Gremlin .NET: Exception from sync > execution gets wrapped in AggregateException > TINKERPOP-1809 -- [TINKERPOP-1809] Process tests shouldn't modify toy data > TINKERPOP-1820 -- [TINKERPOP-1820] Include .NET GLV tests on TravisCI > TINKERPOP-1827 -- [TINKERPOP-1827] Gremlin .NET: Test Suite Runner > TINKERPOP-1831 -- [TINKERPOP-1831] Refactor EventStrategy > TINKERPOP-1836 -- [TINKERPOP-1836] .NET sample project > TINKERPOP-1837 -- [TINKERPOP-1837] Gremlin .NET: Provide type coercion > between IDictionary instances > TINKERPOP-1841 -- [TINKERPOP-1841] Include Python GLV tests on TravisCI > TINKERPOP-1864 -- [TINKERPOP-1864] Gremlin Python tests for GraphSON 2.0 > and 3.0 > TINKERPOP-1878 -- [TINKERPOP-1878] Sparql to Gremlin Compiler > TINKERPOP-1880 -- [TINKERPOP-1880] Gremlin.NET Strong name signature could > not be verified. (HRESULT: 0x80131045) > TINKERPOP-1897 -- [TINKERPOP-1897] Provide Docker images of Gremlin Server > and Console > TINKERPOP-1913 -- [TINKERPOP-1913] Expose metadata from Gremlin Server to > Clients > TINKERPOP-1945 -- [TINKERPOP-1945] Add support for extended GraphSon types > to Gremlin.net > TINKERPOP-1958 -- [TINKERPOP-1958] TinkerGraphCountStrategy can return > wrong counts > TINKERPOP-1959 -- [TINKERPOP-1959] Provide a way to submit scripts to the > server in gremlin-javascript > TINKERPOP-1959-master -- [TINKERPOP-1959] Provide a way to submit scripts > to the server in gremlin-javascript > TINKERPOP-1959-tp33 -- [TINKERPOP-1959] Provide a way to submit scripts to > the server in gremlin-javascript > TINKERPOP-1963 -- [TINKERPOP-1963] Use of reducing step in choose() > TINKERPOP-1967 -- [TINKERPOP-1967] Add a connectedComponent() step > TINKERPOP-1968 -- [TINKERPOP-1968] Refactor elements of Gremlin Server > testing > TINKERPOP-1972 -- [TINKERPOP-1972] inject() tests are throwing exceptions > in .NET GLV tests > TINKERPOP-1972-master -- [TINKERPOP-1972] inject() tests are throwing > exceptions in .NET GLV tests > TINKERPOP-1972-tp33 -- [TINKERPOP-1972] inject() tests are throwing > exceptions in .NET GLV tests > TINKERPOP-1975 -- [TINKERPOP-1975] Introduce with() step modulator > TINKERPOP-1975-x -- [TINKERPOP-1975] Introduce with() step modulator > TINKERPOP-1976 -- [TINKERPOP-1976] Include Computer tests for GLVs > TINKERPOP-1978 -- [TINKERPOP-1978] Check for Websocket connection state > when retrieved from Connection Pool missing > TINKERPOP-1979 -- [TINKERPOP-1979] Several OLAP issues in MathStep > TINKERPOP-1985 -- [TINKERPOP-1985] Update position on bulk loading > TINKERPOP-1986 -- [TINKERPOP-1986] Remove deprecation from > PartitionStrategy, SubgraphStrategy and GremlinScriptEngine > TINKERPOP-1987 -- [TINKERPOP-1987] Bump to Netty 4.1.x > TINKERPOP-1989 -- [TINKERPOP-1989] Preserve order that plugins are applied > in Gremlin Console > TINKERPOP-1990 -- [TINKERPOP-1990] Add a shortestPath() step > TINKERPOP-1996 -- [TINKERPOP-1996] Introduce read() and write() steps > TINKERPOP-1999 -- [TINKERPOP-1999] [Java][gremlin-driver] Query to a remote > server via the websocket client hangs indefinitely if the server becomes > unavailable > TINKERPOP-2011 -- [TINKERPOP-2011] Use NumberHelper on choose() > TINKERPOP-2012 -- [TINKERPOP-2012] Target .NET Standard 2.0 for Gremlin.Net > TINKERPOP-2015 -- [TINKERPOP-2015] Allow users to configure the WebSocket > connections > TINKERPOP-2016 -- [TINKERPOP-2016] Upgrade Jackson FasterXML to 2.9.5 or > later to fix security vulnerability > TINKERPOP-2021 -- [TINKERPOP-2021] Prevent maximum recursion depth failure > TINKERPOP-2023 -- [TINKERPOP-2023] Gremlin Server should not create > self-signed certs > TINKERPOP-2024 -- [TINKERPOP-2024] Gremlin Server Application archetype > should connect via withRemote > TINKERPOP-2025 -- [TINKERPOP-2025] Change to SHA-256/512 and drop SHA-1 for > releases > TINKERPOP-2026 -- [TINKERPOP-2026] Gremlin.Net.Driver should check > ClientWebSocket.State before closing > TINKERPOP-2029 -- [TINKERPOP-2029] Concu
New PMC Member: Jorge Bay-Gondra
Jorge has done a lot of work for TinkerPop with Gremlin.NET and gremlin-javascript. He does really solid code reviews and participates in discussions about the administration of the project itself. It's nice to have him as a part of the TinkerPop Community and it's now nice to have him as a part of the PMC. Welcome, Jorge!
Development branch cleanup
The following branches are scheduled for deletion: TINKERPOP-1342 -- [TINKERPOP-1342] Allow setting scriptEvaluationTimeout in driver TINKERPOP-1365 -- [TINKERPOP-1365] Log the seed used to initialize Random in tests TINKERPOP-1489 -- [TINKERPOP-1489] Provide a Javascript Gremlin Language Variant TINKERPOP-1518 -- [TINKERPOP-1518] Provide a way for providers to expose static Graph.Features to tests TINKERPOP-1595 -- [TINKERPOP-1595] Go through TraversalVertexProgram with a profile and optimize. TINKERPOP-1656 -- [TINKERPOP-1656] Write-time compilation of mutation steps TINKERPOP-1730 -- [TINKERPOP-1730] Gremlin .NET support for GraphSON 3.0 TINKERPOP-1738-wip -- [TINKERPOP-1738] Proper functioning of GraphSONReader depends on order of elements in String representation TINKERPOP-1744 -- [TINKERPOP-1744] Gremlin .NET: Exception from sync execution gets wrapped in AggregateException TINKERPOP-1809 -- [TINKERPOP-1809] Process tests shouldn't modify toy data TINKERPOP-1820 -- [TINKERPOP-1820] Include .NET GLV tests on TravisCI TINKERPOP-1827 -- [TINKERPOP-1827] Gremlin .NET: Test Suite Runner TINKERPOP-1831 -- [TINKERPOP-1831] Refactor EventStrategy TINKERPOP-1836 -- [TINKERPOP-1836] .NET sample project TINKERPOP-1837 -- [TINKERPOP-1837] Gremlin .NET: Provide type coercion between IDictionary instances TINKERPOP-1841 -- [TINKERPOP-1841] Include Python GLV tests on TravisCI TINKERPOP-1864 -- [TINKERPOP-1864] Gremlin Python tests for GraphSON 2.0 and 3.0 TINKERPOP-1878 -- [TINKERPOP-1878] Sparql to Gremlin Compiler TINKERPOP-1880 -- [TINKERPOP-1880] Gremlin.NET Strong name signature could not be verified. (HRESULT: 0x80131045) TINKERPOP-1897 -- [TINKERPOP-1897] Provide Docker images of Gremlin Server and Console TINKERPOP-1913 -- [TINKERPOP-1913] Expose metadata from Gremlin Server to Clients TINKERPOP-1945 -- [TINKERPOP-1945] Add support for extended GraphSon types to Gremlin.net TINKERPOP-1958 -- [TINKERPOP-1958] TinkerGraphCountStrategy can return wrong counts TINKERPOP-1959 -- [TINKERPOP-1959] Provide a way to submit scripts to the server in gremlin-javascript TINKERPOP-1959-master -- [TINKERPOP-1959] Provide a way to submit scripts to the server in gremlin-javascript TINKERPOP-1959-tp33 -- [TINKERPOP-1959] Provide a way to submit scripts to the server in gremlin-javascript TINKERPOP-1963 -- [TINKERPOP-1963] Use of reducing step in choose() TINKERPOP-1967 -- [TINKERPOP-1967] Add a connectedComponent() step TINKERPOP-1968 -- [TINKERPOP-1968] Refactor elements of Gremlin Server testing TINKERPOP-1972 -- [TINKERPOP-1972] inject() tests are throwing exceptions in .NET GLV tests TINKERPOP-1972-master -- [TINKERPOP-1972] inject() tests are throwing exceptions in .NET GLV tests TINKERPOP-1972-tp33 -- [TINKERPOP-1972] inject() tests are throwing exceptions in .NET GLV tests TINKERPOP-1975 -- [TINKERPOP-1975] Introduce with() step modulator TINKERPOP-1975-x -- [TINKERPOP-1975] Introduce with() step modulator TINKERPOP-1976 -- [TINKERPOP-1976] Include Computer tests for GLVs TINKERPOP-1978 -- [TINKERPOP-1978] Check for Websocket connection state when retrieved from Connection Pool missing TINKERPOP-1979 -- [TINKERPOP-1979] Several OLAP issues in MathStep TINKERPOP-1985 -- [TINKERPOP-1985] Update position on bulk loading TINKERPOP-1986 -- [TINKERPOP-1986] Remove deprecation from PartitionStrategy, SubgraphStrategy and GremlinScriptEngine TINKERPOP-1987 -- [TINKERPOP-1987] Bump to Netty 4.1.x TINKERPOP-1989 -- [TINKERPOP-1989] Preserve order that plugins are applied in Gremlin Console TINKERPOP-1990 -- [TINKERPOP-1990] Add a shortestPath() step TINKERPOP-1996 -- [TINKERPOP-1996] Introduce read() and write() steps TINKERPOP-1999 -- [TINKERPOP-1999] [Java][gremlin-driver] Query to a remote server via the websocket client hangs indefinitely if the server becomes unavailable TINKERPOP-2011 -- [TINKERPOP-2011] Use NumberHelper on choose() TINKERPOP-2012 -- [TINKERPOP-2012] Target .NET Standard 2.0 for Gremlin.Net TINKERPOP-2015 -- [TINKERPOP-2015] Allow users to configure the WebSocket connections TINKERPOP-2016 -- [TINKERPOP-2016] Upgrade Jackson FasterXML to 2.9.5 or later to fix security vulnerability TINKERPOP-2021 -- [TINKERPOP-2021] Prevent maximum recursion depth failure TINKERPOP-2023 -- [TINKERPOP-2023] Gremlin Server should not create self-signed certs TINKERPOP-2024 -- [TINKERPOP-2024] Gremlin Server Application archetype should connect via withRemote TINKERPOP-2025 -- [TINKERPOP-2025] Change to SHA-256/512 and drop SHA-1 for releases TINKERPOP-2026 -- [TINKERPOP-2026] Gremlin.Net.Driver should check ClientWebSocket.State before closing TINKERPOP-2029 -- [TINKERPOP-2029] ConcurrentModificationException for InlineFilterStrategy TINKERPOP-2029-master -- [TINKERPOP-2029] ConcurrentModificationException for InlineFilterStrategy TINKERPOP-2030 -- [TINKERPOP-2030] KeepAlive task executed for every Connection.write call TINKERPOP-2031 -- [TINKERPOP-2031] Remove support for -i in gremlin-server.sh TINKERPOP-2032 -- [TINKE
TinkerPop 3.2.10 Released: Nine Inch Gremlins
Hello, TinkerPop 3.2.10 has just been released. This release contains many bug fixes and minor features. Of special note to Javascript developers is the ability to make script-based requests over Gremlin Server protocols. Also, of general note to Gremlin Server user, please pay attention to changes with respect to default security options around SSL. Finally, this release represents the first official release of Docker images for Gremlin Server and Gremlin Console. All of these items are described in greater detail in the Upgrade Documentation linked below. The release artifacts can be found at this location: https://www.apache.org/dyn/closer.lua/tinkerpop/3.2.10/apache-tinkerpop-gremlin-console-3.2.10-bin.zip https://www.apache.org/dyn/closer.lua/tinkerpop/3.2.10/apache-tinkerpop-gremlin-server-3.2.10-bin.zip https://www.apache.org/dyn/closer.lua/tinkerpop/3.2.10/apache-tinkerpop-3.2.10-src.zip The online docs can be found here: http://tinkerpop.apache.org/docs/3.2.10/reference/ (user docs) http://tinkerpop.apache.org/docs/3.2.10/upgrade/#_tinkerpop_3_2_10 (upgrade docs) http://tinkerpop.apache.org/javadocs/3.2.10/core/ (core javadoc) http://tinkerpop.apache.org/javadocs/3.2.10/full/ (full javadoc) The release notes are available here: https://github.com/apache/tinkerpop/blob/3.2.10/CHANGELOG.asciidoc#tinkerpop-3210-release-date-october-15-2018 The Central Maven repo has sync'd as well: https://repo1.maven.org/maven2/org/apache/tinkerpop/tinkerpop/3.2.10/ Python artifacts are available in pypi: https://pypi.python.org/pypi/gremlinpython/3.2.10 .NET artifacts are available in NuGet: https://www.nuget.org/packages/Gremlin.Net/3.2.10 Javascript artifacts are available in npm: https://www.npmjs.com/package/gremlin Docker images for Gremlin Console and Gremlin Server can be found on Docker Hub: https://hub.docker.com/u/tinkerpop/ [image: image.png]
TinkerPop 3.3.4 Released: Gremlin Symphony #40 in G Minor
Hello, TinkerPop 3.3.4 has just been released. Version 3.3.4 contains a number of important changes inherited from 3.2.10, so be sure to review that versions Upgrade Documentation found here: http://tinkerpop.apache.org/docs/3.2.10/upgrade/#_tinkerpop_3_2_10 The release artifacts can be found at this location: https://www.apache.org/dyn/closer.lua/tinkerpop/3.3.4/apache-tinkerpop-gremlin-console-3.3.4-bin.zip https://www.apache.org/dyn/closer.lua/tinkerpop/3.3.4/apache-tinkerpop-gremlin-server-3.3.4-bin.zip https://www.apache.org/dyn/closer.lua/tinkerpop/3.3.4/apache-tinkerpop-3.3.4-src.zip The online docs can be found here: http://tinkerpop.apache.org/docs/3.3.4/reference/ (user docs) http://tinkerpop.apache.org/docs/3.3.4/upgrade/#_tinkerpop_3_3_4 (upgrade docs) http://tinkerpop.apache.org/javadocs/3.3.4/core/ (core javadoc) http://tinkerpop.apache.org/javadocs/3.3.4/full/ (full javadoc) The release notes are available here: https://github.com/apache/tinkerpop/blob/3.3.4/CHANGELOG.asciidoc#tinkerpop-334-release-date-october-15-2018 The Central Maven repo has sync'd as well: https://repo1.maven.org/maven2/org/apache/tinkerpop/tinkerpop/3.3.4/ Python artifacts are available in pypi: https://pypi.python.org/pypi/gremlinpython/3.3.4 .NET artifacts are available in NuGet: https://www.nuget.org/packages/Gremlin.Net/3.3.4 Javascript artifacts are available in npm: https://www.npmjs.com/package/gremlin Docker images for Gremlin Console and Gremlin Server can be found on Docker Hub: https://hub.docker.com/u/tinkerpop/ [image: image.png]
[jira] [Updated] (TINKERPOP-2070) gremlin-javascript: Introduce Connection representation
[
https://issues.apache.org/jira/browse/TINKERPOP-2070?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jorge Bay updated TINKERPOP-2070:
-
Affects Version/s: 3.3.4
> gremlin-javascript: Introduce Connection representation
> ---
>
> Key: TINKERPOP-2070
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2070
> Project: TinkerPop
> Issue Type: Improvement
> Components: javascript
>Affects Versions: 3.3.4
>Reporter: Jorge Bay
>Assignee: Jorge Bay
>Priority: Major
> Fix For: 3.4.0, 3.3.5
>
>
> Currently on gremlin-javascript, {{DriverRemoteConnection}} represents a
> single connection to the server and a {{RemoteConnection}} implementation.
> Like Python and .NET, we need to make the {{DriverRemoteConnection}} use the
> {{Client}} instead of interacting with the websocket directly.
> The dependency tree between classes should be like the following
> {code:java}
> DriverRemoteConnection
> |_ Client
> |_ Connection
> {code}
> These changes don't involve a breaking change, its more of an implementation
> issue. As {{DriverRemoteConnection}} is part of the public API the current
> implementation is a limitation.
> This blocks or at least forces awful workarounds for improvements like
> TINKERPOP-2064.
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[jira] [Updated] (TINKERPOP-2070) gremlin-javascript: Introduce Connection representation
[
https://issues.apache.org/jira/browse/TINKERPOP-2070?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jorge Bay updated TINKERPOP-2070:
-
Fix Version/s: 3.3.5
3.4.0
> gremlin-javascript: Introduce Connection representation
> ---
>
> Key: TINKERPOP-2070
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2070
> Project: TinkerPop
> Issue Type: Improvement
> Components: javascript
>Affects Versions: 3.3.4
>Reporter: Jorge Bay
>Assignee: Jorge Bay
>Priority: Major
> Fix For: 3.4.0, 3.3.5
>
>
> Currently on gremlin-javascript, {{DriverRemoteConnection}} represents a
> single connection to the server and a {{RemoteConnection}} implementation.
> Like Python and .NET, we need to make the {{DriverRemoteConnection}} use the
> {{Client}} instead of interacting with the websocket directly.
> The dependency tree between classes should be like the following
> {code:java}
> DriverRemoteConnection
> |_ Client
> |_ Connection
> {code}
> These changes don't involve a breaking change, its more of an implementation
> issue. As {{DriverRemoteConnection}} is part of the public API the current
> implementation is a limitation.
> This blocks or at least forces awful workarounds for improvements like
> TINKERPOP-2064.
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
