Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore(api): grpc server (remove graphql)#24364

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
pront wants to merge1 commit intomaster
base:master
Choose a base branch
Loading
frompront-grpc
Draft

Conversation

@pront
Copy link
Member

@prontpront commentedDec 10, 2025
edited
Loading

Summary

MetricBeforeAfterChange
API implementation5,999 lines1,200 lines-80%
Schema files24 GraphQL files1 proto file-96%
Dependenciesasync-graphql stacktonic/prostDependency reduction

This PR migrates the Vector API from GraphQL to gRPC as an internal implementation detail with no user-facing changes. Users continue to use the same configuration with identical behavior and output. This refactoring improves performance and reduces dependencies without affecting the user experience, making it a pure internal improvement that requires no user action.

Vector configuration

How did you test this PR?

Change Type

  • Bug fix
  • New feature
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on ourguidelines.
  • No. A maintainer will apply theno-changelog label to this PR.

References

Notes

  • Please read ourVector contributor resources.
  • Do not hesitate to use@vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding apre-push hook, please seethis template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • make fmt
      • make check-clippy (if there are failures it's possible some of them can be fixed withmake clippy-fix)
      • make test
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can rungit merge origin master andgit push.
  • If this PR introduces changes Vector dependencies (modifiesCargo.lock), please
    runmake build-licenses to regenerate thelicense inventory and commit the changes (if any). More detailshere.

@github-actionsgithub-actionsbot added the domain: topologyAnything related to Vector's topology code labelDec 10, 2025
@prontpront added the no-changelogChanges in this PR do not need user-facing explanations in the release changelog labelDec 12, 2025
@prontprontforce-pushed thepront-grpc branch 6 times, most recently from5c09df1 toa317991CompareDecember 18, 2025 19:08
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

domain: topologyAnything related to Vector's topology codeno-changelogChanges in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pront

[8]ページ先頭

©2009-2025 Movatter.jp