Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add NebulaGraph config #5147

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Conversation

GG2002
Copy link
Contributor

@GG2002 GG2002 commented Sep 28, 2024

Which issue does this PR close?

Closes #4553.

Rationale for this change

The NebulaGraph client rust-nebula still needs time to be perfected, so it can merge the NebulaGraph config section first. Ref to #5116.

What changes are included in this PR?

Are there any user-facing changes?

@GG2002 GG2002 requested a review from Xuanwo as a code owner September 28, 2024 16:38
@GG2002 GG2002 force-pushed the add-nebula-graph-config branch from 467b3e0 to 1e0cd77 Compare September 28, 2024 16:39
@GG2002 GG2002 changed the title feat: add NebulaGraph feat: add NebulaGraph config Sep 28, 2024
@GG2002 GG2002 force-pushed the add-nebula-graph-config branch 4 times, most recently from 560cda5 to 92f706c Compare September 28, 2024 17:20
Signed-off-by: feathercyc <feathercyc@163.com>
@GG2002 GG2002 force-pushed the add-nebula-graph-config branch from 92f706c to 5c6c530 Compare September 29, 2024 12:06
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit d1f8f0a into apache:main Oct 1, 2024
227 checks passed
@GG2002 GG2002 deleted the add-nebula-graph-config branch October 1, 2024 15:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for NebulaGraph
2 participants