-
Notifications
You must be signed in to change notification settings - Fork 528
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
feat: use an enhanced CypherAPI to refactor it #2143
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2143 +/- ##
============================================
+ Coverage 61.48% 68.65% +7.16%
- Complexity 484 977 +493
============================================
Files 491 497 +6
Lines 40221 40564 +343
Branches 5615 5662 +47
============================================
+ Hits 24731 27849 +3118
+ Misses 12973 10023 -2950
- Partials 2517 2692 +175
... and 100 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Thanks a lot for your contribution and adopt with the And if possible, give more context (docs/tests/example) about the change/refactor, so that we could review it quickly 👾 @jadepeng You can also help to review the new design |
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherClient.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherClient.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherAPI.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherAPI.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherAPI.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherPlugin.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherOpProcessor.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherOpProcessor.java
Outdated
Show resolved
Hide resolved
...graph-core/src/main/java/org/apache/hugegraph/traversal/optimize/HugePrimaryKeyStrategy.java
Outdated
Show resolved
Hide resolved
...graph-core/src/main/java/org/apache/hugegraph/traversal/optimize/HugePrimaryKeyStrategy.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherClient.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherClient.java
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherClient.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherManager.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherManager.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/gremlin/CypherAPI.java
Outdated
Show resolved
Hide resolved
@shangrilaxie you could try the latest code in And contact us for the
We will improve the way and channel of "contacting us" soon 💬 |
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherClient.java
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherManager.java
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherManager.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherManager.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/api/cypher/CypherManager.java
Outdated
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/auth/StandardAuthenticator.java
Show resolved
Hide resolved
hugegraph-api/src/main/java/org/apache/hugegraph/auth/StandardAuthenticator.java
Outdated
Show resolved
Hide resolved
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.
LGTM
hugegraph-api/src/main/java/org/apache/hugegraph/auth/StandardAuthenticator.java
Outdated
Show resolved
Hide resolved
克隆最新代码,编译打包正常。运行结果:post还是不行,get也不行了。错误信息一样:
返回:(get和post都一样了,原来1.0.0版本的post返回500,get正常) {
"exception": "class jakarta.ws.rs.NotAuthorizedException",
"message": "HTTP 401 Unauthorized",
"cause": ""
} |
@lynnbond could u check the problem or give a short comment for the proper usage? @shangrilaxie please wait author's reply, if we ensure it's a bug, please submit a separate issue |
启动hugeserver,配置好hugegrap图,使用postman发送GET、POST请求。 |
@shangrilaxie 看错误提示,感觉您好像是没有配置 如果使用类 |
I have added a more powerful Cypher API that is enhanced by cypher-gremlin-extensions. The old one has been removed.