Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

search-blitz: adjust limit for snippetAttribution to match gateway#64119

Merged
keegancsmith merged 1 commit into
mainfrom
k/adjust-limit
Jul 29, 2024
Merged

search-blitz: adjust limit for snippetAttribution to match gateway#64119
keegancsmith merged 1 commit into
mainfrom
k/adjust-limit

Conversation

@keegancsmith

Copy link
Copy Markdown
Member

Cody doesn't set a limit, so we use the default that Cody Gateway sets. It uses a value of 4.

Test Plan: yolo

Cody doesn't set a limit, so we use the default that Cody Gateway sets.
It uses a value of 4.

Test Plan: yolo
@keegancsmith keegancsmith requested review from a team and eseliger July 29, 2024 07:08
@cla-bot cla-bot Bot added the cla-signed label Jul 29, 2024
@github-actions github-actions Bot added team/product-platform team/search-platform Issues owned by the search platform team labels Jul 29, 2024
@keegancsmith keegancsmith merged commit e02bd3d into main Jul 29, 2024
@keegancsmith keegancsmith deleted the k/adjust-limit branch July 29, 2024 10:10
@keegancsmith

Copy link
Copy Markdown
Member Author

FYI I will only ship this in a few days, I don't want to muddy the water of guardrails perf work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/product-platform team/search-platform Issues owned by the search platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants