Skip to content
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

fix hugegraph test with apache release 1.2 #5

Merged
merged 62 commits into from
Dec 6, 2023

Conversation

liuxiaocs7
Copy link
Collaborator

@liuxiaocs7 liuxiaocs7 commented Nov 30, 2023

  1. python deploy_start.py all will deploy server and toolchain at the same time.
  2. pytest

Deploy

  • server python deploy_start.py server
  • toolchain python deploy_start.py toolchain

graph function test

graph performance test

None

CI

  • openSource and no auth (except hubble-load)
  • openSource with auth

test data under config/dataset.zip
auth doc: https://hugegraph.apache.org/cn/docs/config/config-authentication/

@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Nov 30, 2023
@liuxiaocs7
Copy link
Collaborator Author

liuxiaocs7 commented Nov 30, 2023

The version of hugegraph-client and hugegraph-server don't match

HG-Client: https://github.com/apache/incubator-hugegraph-toolchain/blob/e8fd3e0a6c10d34b4bf5e92495f493c913d7895e/hugegraph-client/src/main/java/org/apache/hugegraph/driver/HugeClient.java#L110-L114

image

Note: hubble build fails(needs node and yarn)

image

Hubble lang

image

image

image

@liuxiaocs7 liuxiaocs7 changed the title hugegraph test fix hugegraph test with apache release 1.2 Dec 5, 2023
src/config/basic_config.py Outdated Show resolved Hide resolved
src/config/basic_config.py Outdated Show resolved Hide resolved
src/common/deploy_graph.py Outdated Show resolved Hide resolved
src/config/basic_config.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@aroundabout aroundabout left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@liuxiaocs7 liuxiaocs7 merged commit 7af1cab into hugegraph:master Dec 6, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants