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

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

rdtr commented on code in PR #2206:
URL: https://github.com/apache/tinkerpop/pull/2206#discussion_r1303406678


##########
gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/Host.java:
##########
@@ -49,7 +49,8 @@ public final class Host {
     Host(final InetSocketAddress address, final Cluster cluster) {
         this.cluster = cluster;
         this.address = address;
-        this.hostUri = makeUriFromAddress(address, cluster.getPath(), 
cluster.connectionPoolSettings().enableSsl);
+        this.hostUri = makeUriFromAddress(address, cluster.getPath(), 
cluster.connectionPoolSettings().enableSsl,

Review Comment:
   Got it, yeah for explain and profile, supporting that on Neptune side and 
the driver should just add `explain()` or `profile()` in a query, I agree.
   It's too much to support more other endpoints like /status, bulkload for 
Neptune so I don't think at this point adding a custom endpoint is critical at 
least for Neptune.





> Allow gremlin-driver usage over HTTP
> ------------------------------------
>
>                 Key: TINKERPOP-2982
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2982
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: driver, server
>    Affects Versions: 3.6.5
>            Reporter: Stephen Mallette
>            Priority: Major
>
> Allow gremlin-driver to work over HTTP in addition to websockets. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to