-
Notifications
You must be signed in to change notification settings - Fork 424
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GIE Doc] Refine Docs for Cypher #2995
Conversation
@@ -74,10 +74,11 @@ deployment and management of applications. To deploy GIE standalone using Helm, | |||
kubectl describe svc [YOUR_RELEASE_NAME]-gie-standalone-frontend \ | |||
| grep "Endpoints:" | awk -F' ' '{print $2}' | |||
``` | |||
You should see the GIE Frontend service endpoint as `<ip>:<gremlinPort>`. | |||
You should see two exposed endpoints for GIE Frontend service, one is `<ip>:<gremlinPort>` for gremlin querying, another is `<ip>:<cypherPort>` for cypher querying. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Explicitly use two commands, one for obtaining Cypher endpoint, the other for Gremlin endpoint.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2995 +/- ##
===========================================
- Coverage 73.91% 42.38% -31.53%
===========================================
Files 99 99
Lines 10698 10687 -11
===========================================
- Hits 7907 4530 -3377
- Misses 2791 6157 +3366 see 59 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
@@ -102,7 +102,7 @@ You could pass additional key-value pairs to customize the startup configuration | |||
|
|||
```python | |||
# Set the timeout value to 10 min | |||
g = gs.gremlin(graph, params={'pegasus.timeout': 600000}) | |||
g = gs.gremlin(graph, params={'query.execution.timeout.ms': 600000}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How this may be configured with the new python apis?
What do these changes do?
as titled.
Related issue number
Fixes #2987