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.

Sign up for GitHub

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

Jump to bottom

Optimize configuration files to give users more choices #4524

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Optimize configuration files to give users more choices #4524

wants to merge 2 commits into from

Conversation

xinxingi
Copy link

@xinxingi xinxingi commented Jan 17, 2025 •
edited by KevinHuSh
Loading

What problem does this PR solve?

#4523

Type of change

  • Refactoring

@KevinHuSh KevinHuSh added the ci Continue Integration label Jan 17, 2025
@yingfeng
Copy link
Member

CI can not pass due to misconfiguration.

@xinxingi
Copy link
Author

CI can not pass due to misconfiguration.

Sorry, please re-trigger CI

@yingfeng
Copy link
Member

Same, It doesn't work. The es can not be connected after starting up.

@xinxingi
Copy link
Author

Same, It doesn't work. The es can not be connected after starting up.

Haha, sorry sorry, I will do CI after testing the local connectivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci Continue Integration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xinxingi @yingfeng @KevinHuSh