feat(cli): add support for edge_attributes in graph configuration#24593
feat(cli): add support for edge_attributes in graph configuration#24593pront merged 10 commits intovectordotdev:masterfrom
Conversation
|
@vectordev/vector I have also noticed that memory enrichment tables weren't included in the graph output. Not sure if it makes sense to include them, but they have inputs and also outputs sometimes, so it seemed reasonable to add them. |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0964561319
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Codex Review: Didn't find any major issues. Keep it up! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Head branch was pushed to by a user without write access
Summary
Adds support for
edge_attributesbesidesnode_attributesin graph configuration.Vector configuration
The specifics of each component aren't important - just the
graphfields.How did you test this PR?
Ran
vector graphwith the provided file:Change Type
Is this a breaking change?
Does this PR include user facing changes?
no-changeloglabel to this PR.References
Notes
@vectordotdev/vectorto reach out to us regarding this PR.pre-pushhook, please see this template.make fmtmake check-clippy(if there are failures it's possible some of them can be fixed withmake clippy-fix)make testgit merge origin masterandgit push.Cargo.lock), pleaserun
make build-licensesto regenerate the license inventory and commit the changes (if any). More details here.