[jira] [Updated] (TINKERPOP-2098) Improve gremlin-server.sh help output
[ https://issues.apache.org/jira/browse/TINKERPOP-2098?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette updated TINKERPOP-2098: Summary: Improve gremlin-server.sh help output (was: gremlin-server.sh start does not seem to work as advertised) > Improve gremlin-server.sh help output > - > > Key: TINKERPOP-2098 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2098 > Project: TinkerPop > Issue Type: Improvement > Components: server >Affects Versions: 3.3.4 >Reporter: Kelvin R. Lawrence >Priority: Minor > > This may be as simple as a documentation issue but the help text for Gremlin > server implies that I should be able to do: > {code:java} > gremlin-server.sh start conf/myyaml.yaml{code} > However when I do that it does not seem to actually be reading my YAML file > or if it is it is not reading it fully as I am specifying the WsAndHttp > channelizer in the YAML and when the server starts the HTTP endpoint is not > available. > > If I start the server in the terminal with > > {code:java} > gremlin-server.sh conf/myyaml.yaml{code} > > all is fine but I want to start it in the background. > > I spoke to [~pluradj] and he had me try using the environment variable to > specify the YAML file name and that worked with the *start* keyword. > > Is *gremlin-server.sh start conf/myyaml.yam*l supposed to work? > > If not it would be good to change the help text for gremlin-server.sh which > currently reads as follows: > > > {code:java} > Usage: bin/gremlin-server.sh {start|stop|restart|status|console|install > |}{code} > -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Closed] (TINKERPOP-2098) Improve gremlin-server.sh help output
[ https://issues.apache.org/jira/browse/TINKERPOP-2098?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-2098. --- Resolution: Done Assignee: stephen mallette Fix Version/s: 3.4.1 3.3.6 Fixed via CTR on https://github.com/apache/tinkerpop/commit/b8f65835d8c4a50564b3d2c4969287bfd33594ab > Improve gremlin-server.sh help output > - > > Key: TINKERPOP-2098 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2098 > Project: TinkerPop > Issue Type: Improvement > Components: server >Affects Versions: 3.3.4 >Reporter: Kelvin R. Lawrence >Assignee: stephen mallette >Priority: Minor > Fix For: 3.3.6, 3.4.1 > > > This may be as simple as a documentation issue but the help text for Gremlin > server implies that I should be able to do: > {code:java} > gremlin-server.sh start conf/myyaml.yaml{code} > However when I do that it does not seem to actually be reading my YAML file > or if it is it is not reading it fully as I am specifying the WsAndHttp > channelizer in the YAML and when the server starts the HTTP endpoint is not > available. > > If I start the server in the terminal with > > {code:java} > gremlin-server.sh conf/myyaml.yaml{code} > > all is fine but I want to start it in the background. > > I spoke to [~pluradj] and he had me try using the environment variable to > specify the YAML file name and that worked with the *start* keyword. > > Is *gremlin-server.sh start conf/myyaml.yam*l supposed to work? > > If not it would be good to change the help text for gremlin-server.sh which > currently reads as follows: > > > {code:java} > Usage: bin/gremlin-server.sh {start|stop|restart|status|console|install > |}{code} > -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (TINKERPOP-2098) gremlin-server.sh start does not seem to work as advertised
[ https://issues.apache.org/jira/browse/TINKERPOP-2098?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16761271#comment-16761271 ] stephen mallette commented on TINKERPOP-2098: - Updated the help output to be: {code} $ bin/gremlin-server.sh Usage: bin/gremlin-server.sh {start|stop|restart|status|console|install |} startStart the server in the background using conf/gremlin-server.yaml as the default configuration file stop Stop the server restart Stop and start the server status Check if the server is running console Start the server in the foreground using conf/gremlin-server.yaml as the default configuration file install Install dependencies If using a custom YAML configuration file then specify it as the only argument for Gremlin Server to run in the foreground or specify it via the GREMLIN_YAML environment variable. {code} Hope that's clear now. > gremlin-server.sh start does not seem to work as advertised > --- > > Key: TINKERPOP-2098 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2098 > Project: TinkerPop > Issue Type: Improvement > Components: server >Affects Versions: 3.3.4 >Reporter: Kelvin R. Lawrence >Priority: Minor > > This may be as simple as a documentation issue but the help text for Gremlin > server implies that I should be able to do: > {code:java} > gremlin-server.sh start conf/myyaml.yaml{code} > However when I do that it does not seem to actually be reading my YAML file > or if it is it is not reading it fully as I am specifying the WsAndHttp > channelizer in the YAML and when the server starts the HTTP endpoint is not > available. > > If I start the server in the terminal with > > {code:java} > gremlin-server.sh conf/myyaml.yaml{code} > > all is fine but I want to start it in the background. > > I spoke to [~pluradj] and he had me try using the environment variable to > specify the YAML file name and that worked with the *start* keyword. > > Is *gremlin-server.sh start conf/myyaml.yam*l supposed to work? > > If not it would be good to change the help text for gremlin-server.sh which > currently reads as follows: > > > {code:java} > Usage: bin/gremlin-server.sh {start|stop|restart|status|console|install > |}{code} > -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] [tinkerpop] spmallette commented on issue #1056: fix: Path#toString() variable reference bug
Adding typescript definitions is being discussed here: https://issues.apache.org/jira/browse/TINKERPOP-2027 perhaps you could read through that and include any thoughts you might have. i don't think anyone is working on that issue at this time, but you might want to ask on that in any comments you provide. [ Full content available at: https://github.com/apache/tinkerpop/pull/1056 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] thefliik commented on issue #1056: fix: Path#toString() variable reference bug
Seperately, would you be interested in a PR to port the whole javascript library over to typescript? [ Full content available at: https://github.com/apache/tinkerpop/pull/1056 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] thefliik opened pull request #1056: fix: Path#toString() variable reference bug
I'm going through the process of creating typescript definitions for the javascript library, and I noticed a bug in Path#toString(). It incorrectly attempts to call a non-existant local `objects` variable instead of calling the class property `this.objects`. [ Full content available at: https://github.com/apache/tinkerpop/pull/1056 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] jorgebay commented on pull request #1055: TINKERPOP-2150 Support configuring the registry builder in config
Nice catch @dkuppitz , will do! [ Full content available at: https://github.com/apache/tinkerpop/pull/1055 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[jira] [Commented] (TINKERPOP-2151) Support for Vertex and Edge property deserialization in Python GLV
[ https://issues.apache.org/jira/browse/TINKERPOP-2151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760953#comment-16760953 ] stephen mallette commented on TINKERPOP-2151: - No language variant provides properties except for Java which has always been a source of confusion. As a first step toward resolving that for 3.4.0, the decision was made to include {{ReferenceElementStrategy}} which is installed by default in Gremlin Server to detach elements without properties automatically so that the behavior is consistent across all languages including Java. It's just not a good practice to return all properties of an element, anymore than it is a good practice to write {{SELECT * FROM table}}. > As mentioned else where, using `valuesMap` isn't an ideal solution due to the > lack of `inV` and `outV` in the edges. that's a good point. perhaps {{valueMap().with(WithOptions.tokens)}} should be extended a bit for edges somehow [~dkuppitz] ? > Support for Vertex and Edge property deserialization in Python GLV > -- > > Key: TINKERPOP-2151 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2151 > Project: TinkerPop > Issue Type: Improvement > Components: python >Affects Versions: 3.3.5 >Reporter: Matthew Stafford >Priority: Minor > > It seems that the GraphSON serialization in the Python GLV differs from other > GLVs (I've checked JavaScript and assuming Java as well) in that it doesn't > deserialize Vertex and Edge properties. I've found a few discussions of this > (TINKERPOP-1474, TINKERPOP-1565) but it doesn't seem there was a clear > resolution to whether this was something that'd be addressed in the 3.3.x > series or if things would be left with a lack of parity across GLVs and a > different solution was approached in the 3.4.x series. > As mentioned else where, using `valuesMap` isn't an ideal solution due to the > lack of `inV` and `outV` in the edges. Since this was never quite resolved, > does it make sense for me to go in and make the changes to the GraphSON > serializers? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Created] (TINKERPOP-2151) Support for Vertex and Edge property deserialization in Python GLV
Matthew Stafford created TINKERPOP-2151: --- Summary: Support for Vertex and Edge property deserialization in Python GLV Key: TINKERPOP-2151 URL: https://issues.apache.org/jira/browse/TINKERPOP-2151 Project: TinkerPop Issue Type: Improvement Components: python Affects Versions: 3.3.5 Reporter: Matthew Stafford It seems that the GraphSON serialization in the Python GLV differs from other GLVs (I've checked JavaScript and assuming Java as well) in that it doesn't deserialize Vertex and Edge properties. I've found a few discussions of this (TINKERPOP-1474, TINKERPOP-1565) but it doesn't seem there was a clear resolution to whether this was something that'd be addressed in the 3.3.x series or if things would be left with a lack of parity across GLVs and a different solution was approached in the 3.4.x series. As mentioned else where, using `valuesMap` isn't an ideal solution due to the lack of `inV` and `outV` in the edges. Since this was never quite resolved, does it make sense for me to go in and make the changes to the GraphSON serializers? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] [tinkerpop] dkuppitz commented on pull request #1055: TINKERPOP-2150 Support configuring the registry builder in config
The new configuration option should be added to the [docs](https://github.com/apache/tinkerpop/blob/TINKERPOP-2150/docs/src/reference/gremlin-applications.asciidoc#graphbinary). [ Full content available at: https://github.com/apache/tinkerpop/pull/1055 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] dkuppitz commented on issue #1054: TINKERPOP-2149 Add fallback resolver to TypeSerializerRegistry
VOTE +1 [ Full content available at: https://github.com/apache/tinkerpop/pull/1054 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] spmallette commented on issue #1055: TINKERPOP-2150 Support configuring the registry builder in config
Please add an entry to CHANGELOG when you merge but otherwise VOTE +1 [ Full content available at: https://github.com/apache/tinkerpop/pull/1055 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[GitHub] [tinkerpop] spmallette commented on issue #1054: TINKERPOP-2149 Add fallback resolver to TypeSerializerRegistry
Please add an entry to CHANGELOG when you merge, but otherwise VOTE +1 [ Full content available at: https://github.com/apache/tinkerpop/pull/1054 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[jira] [Commented] (TINKERPOP-2150) GraphBinary: Support configuring the TypeSerializerRegistry builder class in config
[ https://issues.apache.org/jira/browse/TINKERPOP-2150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760734#comment-16760734 ] ASF GitHub Bot commented on TINKERPOP-2150: --- jorgebay commented on pull request #1055: TINKERPOP-2150 Support configuring the registry builder in config URL: https://github.com/apache/tinkerpop/pull/1055 https://issues.apache.org/jira/browse/TINKERPOP-2150 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 > GraphBinary: Support configuring the TypeSerializerRegistry builder class in > config > --- > > Key: TINKERPOP-2150 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2150 > Project: TinkerPop > Issue Type: Improvement > Components: io >Affects Versions: 3.4.0 >Reporter: Jorge Bay >Assignee: Jorge Bay >Priority: Major > Fix For: 3.4.1 > > > We currently only support providing custom serializers via configuration. > It would be nice to be able to provide the builder itself, that way providers > can point to a class where all custom and default types are set. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] [tinkerpop] jorgebay opened pull request #1055: TINKERPOP-2150 Support configuring the registry builder in config
https://issues.apache.org/jira/browse/TINKERPOP-2150 VOTE +1 [ Full content available at: https://github.com/apache/tinkerpop/pull/1055 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org
[jira] [Updated] (TINKERPOP-2150) GraphBinary: Support configuring the TypeSerializerRegistry builder class in config
[ https://issues.apache.org/jira/browse/TINKERPOP-2150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jorge Bay updated TINKERPOP-2150: - Component/s: io > GraphBinary: Support configuring the TypeSerializerRegistry builder class in > config > --- > > Key: TINKERPOP-2150 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2150 > Project: TinkerPop > Issue Type: Improvement > Components: io >Affects Versions: 3.4.0 >Reporter: Jorge Bay >Assignee: Jorge Bay >Priority: Major > Fix For: 3.4.1 > > > We currently only support providing custom serializers via configuration. > It would be nice to be able to provide the builder itself, that way providers > can point to a class where all custom and default types are set. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (TINKERPOP-2148) "no connection available!" is being thrown despite lots of free connections
[ https://issues.apache.org/jira/browse/TINKERPOP-2148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760728#comment-16760728 ] Florian Hockmann commented on TINKERPOP-2148: - Unfortunately, this can happen quite easily in 3.4.0 as the driver [first selects|https://github.com/apache/tinkerpop/blob/90c5e2f66c7d6ae15753599144fe6ace78b847e0/gremlin-dotnet/src/Gremlin.Net/Driver/ConnectionPool.cs#L113] the least used connection and then it checks whether it has less than {{MaxInProcessPerConnection}} which results in this exception. If now 2 requests come in in parallel and both get the same connection as the least used one, then you get that exception although other connections are available which haven't reached that limit yet. I guess we could add a dedicated counter on the {{Connection}} object that we increment as soon as the {{Connection}} is selected as the least used one and add some synchronization to prevent race conditions like this. For your current problem, I suggest that you use a much higher limit for {{MaxInProcessPerConnection}}. The default limit of 32 should make this exception a lot less likely to occur. > "no connection available!" is being thrown despite lots of free connections > --- > > Key: TINKERPOP-2148 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2148 > Project: TinkerPop > Issue Type: Bug > Components: dotnet >Affects Versions: 3.4.0 > Environment: Windows 10, C#, .NET Core, CosmosDB Graph >Reporter: Zaoshi >Priority: Minor > > I am submitting multiple graph queries in parallel but in some cases it > starts throwing this exception: > {{Gremlin.Net.Driver.Exceptions.NoConnectionAvailableException : no > connection available!}} > {{ at Gremlin.Net.Driver.ConnectionPool.GetAvailableConnectionAsync()}} > {{ at Gremlin.Net.Driver.GremlinClient.SubmitAsync[T](RequestMessage > requestMessage)}} > When GremlinClient is initialized with \{PoolSize = 1, > MaxInProcessPerConnection = 8} everything works fine, however, with > \{PoolSize = 8, MaxInProcessPerConnection = 1} it starts throwing those > exceptions even though this connection pool should have same throughput. I > have tried to increase pool to \{PoolSize = 128, MaxInProcessPerConnection = > 1} but GremlinClient started to fail even more. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
AW: ATTN Committers/PMC Members: Bio Update
My bio is also still up-to-date. Regards, Florian -Ursprüngliche Nachricht- Von: Robert Dale Gesendet: Sonntag, 3. Februar 2019 03:50 An: dev@tinkerpop.apache.org Betreff: Re: ATTN Committers/PMC Members: Bio Update No change. Thanks. Robert Dale On Fri, Feb 1, 2019 at 4:14 PM Daniel Kuppitz wrote: > I think my bio is still pretty accurate, no changes needed. > > Cheers, > Daniel > > > On Fri, Feb 1, 2019, 1:29 PM Stephen Mallette > wrote: > > > I was hoping that an email to the dev list would have dealt with > > most of this administrative issue, but I haven't seen any updates > > directly to git or in the form of replies as of yet. I will begin > > reaching out directly > to > > folks next week. > > > > On Fri, Jan 25, 2019 at 8:44 AM Stephen Mallette > > > > wrote: > > > > > To Committers/PMC Members, > > > > > > As an Apache TinkerPop committer and/or PMC member, your name is > > > listed > > on > > > the TinkerPop home page in the Contributor List[1] with your > > > "bio". If > > you > > > are active on the project, your "bio" reflects what you have been > working > > > on and what you expect to be working on with respect to TinkerPop > > > for recent times (i.e. for the previous six months and the > > > following six months). If you are currently inactive on the project, your > > > "bio" > > reflects > > > the full scope of all your contributions throughout your active > > periods.[2] > > > > > > Please take a moment to update your bio directly in Git[2] or, if > > > you would prefer, please reply to this email with your bio update > > > and it > will > > > be added for you. If no changes are required, please reply to this > email > > to > > > confirm that this is the case. Feel free to email me directly if > > > you > > like. > > > > > > I think most bios need a bit of a refresh. Jorge, you should > > > definitely add GraphBinary to yours. I think "Release Manager" > > > might be a good one > > for > > > folks to add in given my recent work on the Downloads page. Thanks > > everyone! > > > > > > [1] http://tinkerpop.apache.org/#contributors > > > [2] > > > > > > http://tinkerpop.apache.org/docs/current/dev/developer/#contributor-li > sting > > > [3] > > > > > > https://github.com/apache/tinkerpop/blob/master/docs/site/home/index.h > tml > > > > > >