[jira] [Created] (TINKERPOP-2063) Support for tree() and subgraph() in GLVs

2018-10-11 Thread stephen mallette (JIRA)
stephen mallette created TINKERPOP-2063:
---

 Summary: Support for tree() and subgraph() in GLVs
 Key: TINKERPOP-2063
 URL: https://issues.apache.org/jira/browse/TINKERPOP-2063
 Project: TinkerPop
  Issue Type: Improvement
  Components: dotnet, javascript, python
Affects Versions: 3.3.3
Reporter: stephen mallette


This is a bit of a placeholder to act as a reminder that there still remain 
some functionality that works on the JVM but does not work in GLVs. Need to 
think through ways to deal with that.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (TINKERPOP-2061) Add with() configuration as global to a traversal

2018-10-11 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2061?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646774#comment-16646774
 ] 

ASF GitHub Bot commented on TINKERPOP-2061:
---

dkuppitz 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_r224530299
 
 

 ##
 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:
   Perhaps a little bit more elegant:
   ```java
   public default TraversalSource with(final String key, final Object 
value) {
   final OptionsStrategy.Builder builder = OptionsStrategy.build();
   getStrategies().getStrategy(OptionsStrategy.class)
   .ifPresent(optionsStrategy -> 
optionsStrategy.getOptions().forEach(builder::with));
   builder.with(key, value);
   return withStrategies(builder.create());
   }
   ```


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
>
> 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] dkuppitz commented on pull request #961: TINKERPOP-2061 Added with() option for traversal configuration

2018-10-11 Thread GitHub
Perhaps a little bit more elegant:
```java
public default TraversalSource with(final String key, final Object value) {
final OptionsStrategy.Builder builder = OptionsStrategy.build();
getStrategies().getStrategy(OptionsStrategy.class)
.ifPresent(optionsStrategy -> 
optionsStrategy.getOptions().forEach(builder::with));
builder.with(key, value);
return withStrategies(builder.create());
}
```

[ Full content available at: https://github.com/apache/tinkerpop/pull/961 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org


[jira] [Commented] (TINKERPOP-2060) Make Mutating steps non-final

2018-10-11 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646749#comment-16646749
 ] 

ASF GitHub Bot commented on TINKERPOP-2060:
---

dkuppitz commented on issue #960: TINKERPOP-2060 Removed final modifier on 
Mutating steps
URL: https://github.com/apache/tinkerpop/pull/960#issuecomment-429033204
 
 
   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


> Make Mutating steps non-final
> -
>
> Key: TINKERPOP-2060
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2060
> 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
>
>
> This has been discussed a few times now over the years with no real reason to 
> not allow for it ever posed that I can remember. Graph providers could save a 
> fair bit of trouble in developing custom steps if they could extend the 
> {{Mutating}} steps. Graph providers will need to take the burden of the 
> potential for breaking changes that might occur as a result of these steps 
> changing though.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [tinkerpop] dkuppitz commented on issue #960: TINKERPOP-2060 Removed final modifier on Mutating steps

2018-10-11 Thread GitHub
VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/960 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org


[jira] [Comment Edited] (TINKERPOP-2062) Add Traversal class to CoreImports

2018-10-11 Thread Daniel Kuppitz (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646628#comment-16646628
 ] 

Daniel Kuppitz edited comment on TINKERPOP-2062 at 10/11/18 3:35 PM:
-

I tried, but couldn't figure out how.


was (Author: dkuppitz):
I tried, but couldn't figure out why.

> Add Traversal class to CoreImports
> --
>
> Key: TINKERPOP-2062
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2062
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.3.3
>Reporter: Daniel Kuppitz
>Assignee: Daniel Kuppitz
>Priority: Minor
>
> Apparently, some people like to copy Gremlin queries they've written in Java 
> and paste them into the Gremlin console. This should usually work pretty 
> well, however, there are some oddities in particular with {{union()}} step 
> which requires you to cast the child traversals to {{Traversal}} if they have 
> different return types, e.g.:
> {noformat}
> g.V().
> union(
>   __.identity(),
>   (Traversal)__.outE()
> )
> {noformat}
> This works in a local console session but fails if the query is sent to a 
> server as {{Traversal}} is not part of the core imports. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (TINKERPOP-2062) Add Traversal class to CoreImports

2018-10-11 Thread Daniel Kuppitz (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646628#comment-16646628
 ] 

Daniel Kuppitz commented on TINKERPOP-2062:
---

I tried, but couldn't figure out why.

> Add Traversal class to CoreImports
> --
>
> Key: TINKERPOP-2062
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2062
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.3.3
>Reporter: Daniel Kuppitz
>Assignee: Daniel Kuppitz
>Priority: Minor
>
> Apparently, some people like to copy Gremlin queries they've written in Java 
> and paste them into the Gremlin console. This should usually work pretty 
> well, however, there are some oddities in particular with {{union()}} step 
> which requires you to cast the child traversals to {{Traversal}} if they have 
> different return types, e.g.:
> {noformat}
> g.V().
> union(
>   __.identity(),
>   (Traversal)__.outE()
> )
> {noformat}
> This works in a local console session but fails if the query is sent to a 
> server as {{Traversal}} is not part of the core imports. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Closed] (TINKERPOP-2043) CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and doesn't recover

2018-10-11 Thread Florian Hockmann (JIRA)


 [ 
https://issues.apache.org/jira/browse/TINKERPOP-2043?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Florian Hockmann closed TINKERPOP-2043.
---
Resolution: Duplicate

> CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and 
> doesn't recover
> 
>
> Key: TINKERPOP-2043
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2043
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.4.0
> Environment: Gremlin.Net 3.4.0-rc1
> Microsoft.NetCore.App v2.0
>Reporter: Sami
>Priority: Critical
>
> We have a .Net Core web api with an Azure Cosmos db backend.
> Our data access layer calls 
> _gremlinConnection.Client.SubmitAsync(query) using a singleton 
> gremlin connection. This has worked well except when the API is called with 
> many parallel web api requests. The endpoints that we provide make around 5 
> gremlin calls (some to read, some to write cosmos db edges). 
> During performance testing we noticed System.Net.Http.WinHttpException
> Looking into the issue, we found that NrConnections was at 96 and the stack 
> trace showed that Gremlin.Net called 
> ConnectionPool.CloseAndRemoveAllConnections. 
> We would expect that Gremlin.Net would recover after a call to 
> CloseAndRemoveAllConnections and finish its "Client.SubmitAsync" call but 
> instead it bubbles up the System.Net.Http.WinHttpException and immediately 
> following calls to our API failed with the same 
> System.Net.Http.WinHttpException.
>  
> Would you agree that Gremlin.Net's 
> ConnectionPool.CloseAndRemoveAllConnections should recover on its own and 
> +complete+ its Client.SubmitAsync call or do you expect the consumer of 
> Gremlin.Net to provide some work-around? If so, how can the pool be 
> re-initiated or reset from outside of the Gremlin.Net library?
>  
> Here is the stack trace. Please note that we are ready for a release and this 
> is holding us back. By the way, we do use Gremlin.Net 3.4 RC1 but 3.3.3 also 
> proved to have the same issue.
>  
> {code:java}
> // code placeholder
> ProjectX.Exceptions.PersistenceFailedException:
> at ProjectX.Repository.Cosmos.UnitOfWork+d__32.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.GetResult 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> ProjectX.ModelAPI.Controllers.PersonsController+d__7.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__12.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__10.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow 
> (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next 
> (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at 
> 

[jira] [Commented] (TINKERPOP-2043) CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and doesn't recover

2018-10-11 Thread Florian Hockmann (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646623#comment-16646623
 ] 

Florian Hockmann commented on TINKERPOP-2043:
-

However, the fix also went into the release branches for the versions 3.2.10 
and 3.3.4 which we expect to release relatively soon. So, you can just wait for 
the release of 3.3.4 and then use that version.

Anyway, thanks for reporting back on this. I guess we can now close this 
ticket. You can re-open or comment here again in case you see this problem 
again with 3.4.0-rc2 or the to be released version 3.3.4.

> CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and 
> doesn't recover
> 
>
> Key: TINKERPOP-2043
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2043
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.4.0
> Environment: Gremlin.Net 3.4.0-rc1
> Microsoft.NetCore.App v2.0
>Reporter: Sami
>Priority: Critical
>
> We have a .Net Core web api with an Azure Cosmos db backend.
> Our data access layer calls 
> _gremlinConnection.Client.SubmitAsync(query) using a singleton 
> gremlin connection. This has worked well except when the API is called with 
> many parallel web api requests. The endpoints that we provide make around 5 
> gremlin calls (some to read, some to write cosmos db edges). 
> During performance testing we noticed System.Net.Http.WinHttpException
> Looking into the issue, we found that NrConnections was at 96 and the stack 
> trace showed that Gremlin.Net called 
> ConnectionPool.CloseAndRemoveAllConnections. 
> We would expect that Gremlin.Net would recover after a call to 
> CloseAndRemoveAllConnections and finish its "Client.SubmitAsync" call but 
> instead it bubbles up the System.Net.Http.WinHttpException and immediately 
> following calls to our API failed with the same 
> System.Net.Http.WinHttpException.
>  
> Would you agree that Gremlin.Net's 
> ConnectionPool.CloseAndRemoveAllConnections should recover on its own and 
> +complete+ its Client.SubmitAsync call or do you expect the consumer of 
> Gremlin.Net to provide some work-around? If so, how can the pool be 
> re-initiated or reset from outside of the Gremlin.Net library?
>  
> Here is the stack trace. Please note that we are ready for a release and this 
> is holding us back. By the way, we do use Gremlin.Net 3.4 RC1 but 3.3.3 also 
> proved to have the same issue.
>  
> {code:java}
> // code placeholder
> ProjectX.Exceptions.PersistenceFailedException:
> at ProjectX.Repository.Cosmos.UnitOfWork+d__32.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.GetResult 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> ProjectX.ModelAPI.Controllers.PersonsController+d__7.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__12.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__10.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> 

[jira] [Comment Edited] (TINKERPOP-2043) CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and doesn't recover

2018-10-11 Thread Sami (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646567#comment-16646567
 ] 

Sami edited comment on TINKERPOP-2043 at 10/11/18 3:12 PM:
---

We've upgraded to 3.4.0-rc2. That branch seems to have solved the issue because 
we no longer get those exceptions, Florian.

We're about to release end of October. How soon would 3.4 be ready for 
production?

Sami

 


was (Author: samimajed):
We've upgraded to 3.4.0-rc2 and it seemed to have solved the problem. We're 
about to release in the next two weeks. Would 3.4 be production ready soon?

Sami

 

> CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and 
> doesn't recover
> 
>
> Key: TINKERPOP-2043
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2043
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.4.0
> Environment: Gremlin.Net 3.4.0-rc1
> Microsoft.NetCore.App v2.0
>Reporter: Sami
>Priority: Critical
>
> We have a .Net Core web api with an Azure Cosmos db backend.
> Our data access layer calls 
> _gremlinConnection.Client.SubmitAsync(query) using a singleton 
> gremlin connection. This has worked well except when the API is called with 
> many parallel web api requests. The endpoints that we provide make around 5 
> gremlin calls (some to read, some to write cosmos db edges). 
> During performance testing we noticed System.Net.Http.WinHttpException
> Looking into the issue, we found that NrConnections was at 96 and the stack 
> trace showed that Gremlin.Net called 
> ConnectionPool.CloseAndRemoveAllConnections. 
> We would expect that Gremlin.Net would recover after a call to 
> CloseAndRemoveAllConnections and finish its "Client.SubmitAsync" call but 
> instead it bubbles up the System.Net.Http.WinHttpException and immediately 
> following calls to our API failed with the same 
> System.Net.Http.WinHttpException.
>  
> Would you agree that Gremlin.Net's 
> ConnectionPool.CloseAndRemoveAllConnections should recover on its own and 
> +complete+ its Client.SubmitAsync call or do you expect the consumer of 
> Gremlin.Net to provide some work-around? If so, how can the pool be 
> re-initiated or reset from outside of the Gremlin.Net library?
>  
> Here is the stack trace. Please note that we are ready for a release and this 
> is holding us back. By the way, we do use Gremlin.Net 3.4 RC1 but 3.3.3 also 
> proved to have the same issue.
>  
> {code:java}
> // code placeholder
> ProjectX.Exceptions.PersistenceFailedException:
> at ProjectX.Repository.Cosmos.UnitOfWork+d__32.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.GetResult 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> ProjectX.ModelAPI.Controllers.PersonsController+d__7.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__12.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__10.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, 

[jira] [Commented] (TINKERPOP-2043) CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and doesn't recover

2018-10-11 Thread stephen mallette (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646572#comment-16646572
 ] 

stephen mallette commented on TINKERPOP-2043:
-

We've delayed a full 3.4.0 for a couple of months. I don't think we'll release 
it until the end of the year.

> CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and 
> doesn't recover
> 
>
> Key: TINKERPOP-2043
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2043
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.4.0
> Environment: Gremlin.Net 3.4.0-rc1
> Microsoft.NetCore.App v2.0
>Reporter: Sami
>Priority: Critical
>
> We have a .Net Core web api with an Azure Cosmos db backend.
> Our data access layer calls 
> _gremlinConnection.Client.SubmitAsync(query) using a singleton 
> gremlin connection. This has worked well except when the API is called with 
> many parallel web api requests. The endpoints that we provide make around 5 
> gremlin calls (some to read, some to write cosmos db edges). 
> During performance testing we noticed System.Net.Http.WinHttpException
> Looking into the issue, we found that NrConnections was at 96 and the stack 
> trace showed that Gremlin.Net called 
> ConnectionPool.CloseAndRemoveAllConnections. 
> We would expect that Gremlin.Net would recover after a call to 
> CloseAndRemoveAllConnections and finish its "Client.SubmitAsync" call but 
> instead it bubbles up the System.Net.Http.WinHttpException and immediately 
> following calls to our API failed with the same 
> System.Net.Http.WinHttpException.
>  
> Would you agree that Gremlin.Net's 
> ConnectionPool.CloseAndRemoveAllConnections should recover on its own and 
> +complete+ its Client.SubmitAsync call or do you expect the consumer of 
> Gremlin.Net to provide some work-around? If so, how can the pool be 
> re-initiated or reset from outside of the Gremlin.Net library?
>  
> Here is the stack trace. Please note that we are ready for a release and this 
> is holding us back. By the way, we do use Gremlin.Net 3.4 RC1 but 3.3.3 also 
> proved to have the same issue.
>  
> {code:java}
> // code placeholder
> ProjectX.Exceptions.PersistenceFailedException:
> at ProjectX.Repository.Cosmos.UnitOfWork+d__32.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.GetResult 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> ProjectX.ModelAPI.Controllers.PersonsController+d__7.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__12.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__10.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow 
> (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next 
> (Microsoft.AspNetCore.Mvc.Core, 

[jira] [Commented] (TINKERPOP-2062) Add Traversal class to CoreImports

2018-10-11 Thread stephen mallette (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646569#comment-16646569
 ] 

stephen mallette commented on TINKERPOP-2062:
-

Should we attempt to fix the problem at its source - in other words fix the 
{{union()}} signature?

> Add Traversal class to CoreImports
> --
>
> Key: TINKERPOP-2062
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2062
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.3.3
>Reporter: Daniel Kuppitz
>Assignee: Daniel Kuppitz
>Priority: Minor
>
> Apparently, some people like to copy Gremlin queries they've written in Java 
> and paste them into the Gremlin console. This should usually work pretty 
> well, however, there are some oddities in particular with {{union()}} step 
> which requires you to cast the child traversals to {{Traversal}} if they have 
> different return types, e.g.:
> {noformat}
> g.V().
> union(
>   __.identity(),
>   (Traversal)__.outE()
> )
> {noformat}
> This works in a local console session but fails if the query is sent to a 
> server as {{Traversal}} is not part of the core imports. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (TINKERPOP-2043) CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and doesn't recover

2018-10-11 Thread Sami (JIRA)


[ 
https://issues.apache.org/jira/browse/TINKERPOP-2043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16646567#comment-16646567
 ] 

Sami commented on TINKERPOP-2043:
-

We've upgraded to 3.4.0-rc2 and it seemed to have solved the problem. We're 
about to release in the next two weeks. Would 3.4 be production ready soon?

Sami

 

> CloseAndRemoveAllConnections bubbles up System.Net.Http.WinHttpException and 
> doesn't recover
> 
>
> Key: TINKERPOP-2043
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2043
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.4.0
> Environment: Gremlin.Net 3.4.0-rc1
> Microsoft.NetCore.App v2.0
>Reporter: Sami
>Priority: Critical
>
> We have a .Net Core web api with an Azure Cosmos db backend.
> Our data access layer calls 
> _gremlinConnection.Client.SubmitAsync(query) using a singleton 
> gremlin connection. This has worked well except when the API is called with 
> many parallel web api requests. The endpoints that we provide make around 5 
> gremlin calls (some to read, some to write cosmos db edges). 
> During performance testing we noticed System.Net.Http.WinHttpException
> Looking into the issue, we found that NrConnections was at 96 and the stack 
> trace showed that Gremlin.Net called 
> ConnectionPool.CloseAndRemoveAllConnections. 
> We would expect that Gremlin.Net would recover after a call to 
> CloseAndRemoveAllConnections and finish its "Client.SubmitAsync" call but 
> instead it bubbles up the System.Net.Http.WinHttpException and immediately 
> following calls to our API failed with the same 
> System.Net.Http.WinHttpException.
>  
> Would you agree that Gremlin.Net's 
> ConnectionPool.CloseAndRemoveAllConnections should recover on its own and 
> +complete+ its Client.SubmitAsync call or do you expect the consumer of 
> Gremlin.Net to provide some work-around? If so, how can the pool be 
> re-initiated or reset from outside of the Gremlin.Net library?
>  
> Here is the stack trace. Please note that we are ready for a release and this 
> is holding us back. By the way, we do use Gremlin.Net 3.4 RC1 but 3.3.3 also 
> proved to have the same issue.
>  
> {code:java}
> // code placeholder
> ProjectX.Exceptions.PersistenceFailedException:
> at ProjectX.Repository.Cosmos.UnitOfWork+d__32.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.GetResult 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> ProjectX.ModelAPI.Controllers.PersonsController+d__7.MoveNext 
> (<...removed...>)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification
>  (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__12.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at 
> Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__10.MoveNext
>  (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw 
> (System.Private.CoreLib, Version=4.0.0.0, Culture=neutral, 
> PublicKeyToken=7cec85d7bea7798e)
> at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow 
> (Microsoft.AspNetCore.Mvc.Core, Version=2.0.2.0, Culture=neutral, 
> PublicKeyToken=adb9793829ddae60)
> at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next 
> 

[jira] [Created] (TINKERPOP-2062) Add Traversal class to CoreImports

2018-10-11 Thread Daniel Kuppitz (JIRA)
Daniel Kuppitz created TINKERPOP-2062:
-

 Summary: Add Traversal class to CoreImports
 Key: TINKERPOP-2062
 URL: https://issues.apache.org/jira/browse/TINKERPOP-2062
 Project: TinkerPop
  Issue Type: Improvement
  Components: server
Affects Versions: 3.3.3
Reporter: Daniel Kuppitz
Assignee: Daniel Kuppitz


Apparently, some people like to copy Gremlin queries they've written in Java 
and paste them into the Gremlin console. This should usually work pretty well, 
however, there are some oddities in particular with {{union()}} step which 
requires you to cast the child traversals to {{Traversal}} if they have 
different return types, e.g.:

{noformat}
g.V().
union(
  __.identity(),
  (Traversal)__.outE()
)
{noformat}

This works in a local console session but fails if the query is sent to a 
server as {{Traversal}} is not part of the core imports. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


AW: [DISCUSS] Add Gremlin IDE to Provider Index

2018-10-11 Thread Florian Hockmann
Big +1 from my side and thanks for your work on this tool, Dave!

A tool to visualize the results of Gremlin traversals is the most important 
thing that was missing (apart from GremlinBin which seems to be offline) in the 
TinkerPop ecosystem from my point of view.

-Ursprüngliche Nachricht-
Von: Stephen Mallette  
Gesendet: Donnerstag, 11. Oktober 2018 13:53
An: dev@tinkerpop.apache.org
Betreff: [DISCUSS] Add Gremlin IDE to Provider Index

Gremlin IDe has announced a new release. I was a bit surprised that we didn't 
have it in the Powered By section yet:

https://github.com/bechbd/gremlin-ide

Unless there are objections in the next 72 hours, I will assume lazy consensus 
and move forward to get this added.



[DISCUSS] Add Gremlin IDE to Provider Index

2018-10-11 Thread Stephen Mallette
Gremlin IDe has announced a new release. I was a bit surprised that we
didn't have it in the Powered By section yet:

https://github.com/bechbd/gremlin-ide

Unless there are objections in the next 72 hours, I will assume lazy
consensus and move forward to get this added.