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.

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

Vectorize: optionally return metadata for a vector query #1297

Merged
merged 2 commits into from
Oct 31, 2023
Merged

Vectorize: optionally return metadata for a vector query #1297

merged 2 commits into from
Oct 31, 2023

Conversation

ndisidore
Copy link
Contributor

This introduces a breaking change to the Vectorize query API that changes both the request and response formats.
This is managed by a new compatibility flag which is forwarded to the implementation to derive the correct format.

@irvinebroque
Copy link
Collaborator

Nice! Can we make sure to prep a docs PR in advance of this going out? (https://developers.cloudflare.com/workers/configuration/compatibility-dates/#compatibility-flags) — thanks!

@ndisidore
Copy link
Contributor Author

Nice! Can we make sure to prep a docs PR in advance of this going out? (https://developers.cloudflare.com/workers/configuration/compatibility-dates/#compatibility-flags) — thanks!

Gotcha 👍
cloudflare/cloudflare-docs#11306

@jasnell
Copy link
Member

jasnell commented Oct 17, 2023

Marking this as "needs-internal-pr" just to make sure there is an internal CI run done before this lands. Once/if the internal PR and CI is done and everything looks good, the label can be removed and this can be landed.

@jasnell
Copy link
Member

jasnell commented Oct 31, 2023

@mrbbot ... looks like this needs a sign off from someone on the wrangler team since it touches one of the CODEOWNERs rules.

Copy link
Contributor

@mrbbot mrbbot left a comment

Choose a reason for hiding this comment

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

Rubber stamp as I'm on my phone. We should move the supported compatibility date into its own file and import it, so compatibility flag additions don't require Wrangler approval. 👍

@jasnell jasnell merged commit ce4237a into cloudflare:main Oct 31, 2023
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants