-
Notifications
You must be signed in to change notification settings - Fork 763
Port count datapoints query to gateway #5115
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ca7eda3 to
9608492
Compare
f65a7c3 to
0fd57e3
Compare
9608492 to
6115e2a
Compare
7c82b61 to
43a9275
Compare
0fd57e3 to
4caec22
Compare
43a9275 to
967fb6c
Compare
0ef040c to
f31467c
Compare
Base automatically changed from
sl/port-count-inference-by-variant-clickhouse
to
main
December 11, 2025 21:08
4caec22 to
4fc31d9
Compare
8a2cacb to
461317d
Compare
461317d to
f9ce4f4
Compare
f9ce4f4 to
c334ad0
Compare
GabrielBianconi
previously approved these changes
Dec 12, 2025
Count datapoints typescript
2da1d99 to
df0291a
Compare
Member
Author
|
Merge conflict free to stamp |
Member
Author
|
/merge-queue |
Contributor
|
π Merge queue workflow triggered! View the run: https://github.com/tensorzero/tensorzero/actions/runs/20186292155 |
GabrielBianconi
approved these changes
Dec 13, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a new route
GET /internal/datasets/{dataset_name}/datapoints/count?{function_name=...}that returns the number of datapoints in a given dataset, optionally with a function. This replaces the old napicount_datapoints_for_dataset_functionthat also requires taking a function type.A step towards #5062.
Important
Adds a new route to count datapoints in a dataset, replacing deprecated functionality, with updated handlers and tests.
GET /internal/datasets/{dataset_name}/datapoints/countroute ininternal.rsto return datapoint count, optionally filtered by function name.count_datapoints_for_dataset_functionwithcount_datapoints_for_datasetindataset_queries.rs.get_datapoint_count_handleringet_datapoint_count.rs.datapoint_count.rsto verify new route functionality.CountDatapointsForDatasetFunctionParams.tsand related code.TensorZeroClientintensorzero.tsto use new route for datapoint count.routes.ts.This description was created by
for df0291a. You can customize this summary. It will automatically update as commits are pushed.