Skip to content

feat/apis: add algorithm-specific payload options - #3589

Draft
Matts966 wants to merge 6 commits into
mainfrom
feature/grpc/common-any-args
Draft

Matts966 wants to merge 6 commits into
mainfrom
feature/grpc/common-any-args

Conversation

@Matts966

@Matts966 Matts966 commented Aug 5, 2026 •

Copy link
Copy Markdown
Member

Summary

  • add algorithm-specific google.protobuf.Any options to Search and Insert payload configs
  • preserve existing Remove timestamp enum identifiers in generated Go code
  • initialize the new Insert options field in Rust upsert conversions
  • keep all ZVec-specific message definitions and runtime changes out of this PR

Verification

  • make proto/all (Buf generation succeeded; repository replacements were applied in smaller batches because macOS xargs exceeded its command-line limit)
  • make license
  • GitHub Build protobuf check passed on the rebased head

Stacked PR

#3574 is based on feature/grpc/common-any-args and contains the ZVec-specific messages and implementation.

@coderabbitai

coderabbitai Bot commented Aug 5, 2026 •

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@vdaas-ci

vdaas-ci commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • ✅ /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-v2 - run E2E deploy & integration test (tests/v2/e2e)
@Matts966

Matts966 commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

vdaas-ci commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 7, 2026 •

Copy link
Copy Markdown

Deploying vald with  Cloudflare Pages  Cloudflare Pages

Latest commit: 469364f
Status: ✅  Deploy successful!
Preview URL: https://12648772.vald.pages.dev
Branch Preview URL: https://feature-grpc-common-any-args.vald.pages.dev

View logs

@vdaas-ci

vdaas-ci commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

[FORMAT] Failed to format.

@Matts966

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

@Matts966

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Failed to format.

@github-actions github-actions Bot added size/XL and removed size/XL labels Aug 12, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

golangci

🚫 [golangci] reported by reviewdog 🐶
Magic number: 11, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 12, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 13, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 14, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 15, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 16, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 17, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 18, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 19, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 20, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 21, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 22, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 23, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 24, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 25, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 26, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 27, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 28, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
ST1006: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (staticcheck)

func (this *ZVec_DocumentOptions) EqualVT(that *ZVec_DocumentOptions) bool {


🚫 [golangci] reported by reviewdog 🐶
ST1006: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (staticcheck)

func (this *ZVec_DocumentOptions) EqualMessageVT(thatMsg proto.Message) bool {


🚫 [golangci] reported by reviewdog 🐶
unnecessary conversion (unconvert)

binary.LittleEndian.PutUint32(dAtA[i:], uint32(f1))

@Matts966

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

@github-actions github-actions Bot added size/XL and removed size/XL labels Aug 12, 2026
@Matts966
Matts966 force-pushed the feature/grpc/common-any-args branch from ee05196 to 469364f Compare September 30, 2026 04:03
@Matts966 Matts966 mentioned this pull request Sep 30, 2026
@github-actions github-actions Bot removed the size/M label Sep 30, 2026
@@ -19398,6 +19362,40 @@ func (m *Search_Config) UnmarshalVT(dAtA []byte) error {
break
}
}
case 13:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [golangci] reported by reviewdog 🐶
19365-19398 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:20617-20650 (dupl)

@@ -20628,6 +20614,40 @@ func (m *Insert_Config) UnmarshalVT(dAtA []byte) error {
break
}
}
case 4:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [golangci] reported by reviewdog 🐶
20617-20650 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:31547-31580 (dupl)

@@ -31610,6 +31544,40 @@ func (m *Search_Config) UnmarshalVTUnsafe(dAtA []byte) error {
break
}
}
case 13:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [golangci] reported by reviewdog 🐶
31547-31580 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:32807-32840 (dupl)

@@ -32848,6 +32804,40 @@ func (m *Insert_Config) UnmarshalVTUnsafe(dAtA []byte) error {
break
}
}
case 4:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [golangci] reported by reviewdog 🐶
32807-32840 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:19365-19398 (dupl)

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment